Installed node settings hook

[hardillb]

fixes #7736

Description

Uses DB hooks to keep latest version of Node-RED nodes list up to date

This PR merges into #7772

Related Issue(s)

#7736

Checklist

• I have read the contribution guidelines
• Suitable unit/system level tests have been added and they pass
• Documentation has been updated
  • Upgrade instructions
  • Configuration details
  • Concepts
• Changes flowforge.yml?
  • Issue/PR raised on FlowFuse/helm to update ConfigMap Template
  • Issue/PR raised on FlowFuse/CloudProject to update values for Staging/Production
• Link to Changelog Entry PR, or note why one is not needed.

Labels

• Includes a DB migration? -> add the area:migration label

[codecov]

Codecov Report

❌ Patch coverage is 68.75000% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.54%. Comparing base (f6381d8) to head (2794982).

Files with missing lines	Patch %	Lines
forge/db/models/StorageSettings.js	76.92%	3 Missing ⚠️
forge/ee/lib/bom/tasks/cache-catalogues.js	0.00%	3 Missing ⚠️
forge/db/models/NodeREDNodeVersions.js	77.77%	2 Missing ⚠️
...igrations/20260717-01-add-node-red-node-version.js	80.00%	1 Missing ⚠️
forge/db/models/Project.js	50.00%	1 Missing ⚠️

Additional details and impacted files

@@                     Coverage Diff                      @@
##           cache-catalogue-versions    #7880      +/-   ##
============================================================
- Coverage                     75.55%   75.54%   -0.01%     
============================================================
  Files                           434      436       +2     
  Lines                         23064    23096      +32     
  Branches                       6112     6115       +3     
============================================================
+ Hits                          17425    17447      +22     
- Misses                         5639     5649      +10     

Flag	Coverage Δ
backend	75.54% <68.75%> (-0.01%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[Steve-Mcl]

I see I was tagged as reviewer for this but Is it ready for review Ben? Just from a very quick glance I see things like swallowed errors (// console.log(err)), no support for devices (even though associations to device is setup), no tests, and the target branch PR is still in draft/has no reviewer. 🤷

note also, I believe you might need to drop the association for device and switch to application logic as we have done for things like access_token (casting from int to string for the device.id/ownerId will likely fail for PG) - but in saying that I see existing tests did not fail

[hardillb]

I see I was tagged as reviewer for this but Is it ready for review Ben? Just from a very quick glance I see things like swallowed errors (// console.log(err)), no support for devices (even though associations to device is setup), no tests, and the target branch PR is still in draft/has no reviewer. 🤷

Yes all intentional, especially the devices as we have no way to gather that information as it is not shared. The current table is built that way to make it work if/when it ever is available.

The errors get dropped because this is a async hook fired in the background and the only possible errors are back end db so failing to update is a small problem

[knolleary]

I doubt the guard is necessary as (unlike MCP) this model is in the core set and will always be available. But we do need to fix the reference...

Suggested change

	if (app.db.models.NodeREDNodeVersions?.destory) {
	await app.db.models.destory({
	if (app.db.models.NodeREDNodeVersions?.destory) {
	await app.db.models.NodeREDNodeVersions.destory({