Move round feature enablement

[hardillb]

part of FlowFuse/engineering#230

Description

Adds new license tiers and enables features accordingly as laid out in FlowFuse/engineering#230.

The enterprise tier should continue to add all new features so they are covered by tests

Related Issue(s)

FlowFuse/engineering#230

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 35.49784% with 149 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.79%. Comparing base (a6f7614) to head (78d997e).
⚠️ Report is 39 commits behind head on main.

Files with missing lines	Patch %	Lines
forge/licensing/license-generator.js	0.00%	60 Missing ⚠️
forge/licensing/license-generator-old.js	0.00%	59 Missing ⚠️
forge/ee/routes/index.js	65.95%	16 Missing ⚠️
forge/ee/lib/index.js	78.18%	12 Missing ⚠️
forge/licensing/index.js	50.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #8155      +/-   ##
==========================================
- Coverage   76.19%   75.79%   -0.41%     
==========================================
  Files         443      446       +3     
  Lines       23726    24108     +382     
  Branches     6314     6431     +117     
==========================================
+ Hits        18079    18272     +193     
- Misses       5647     5836     +189     

Flag	Coverage Δ
backend	75.79% <35.49%> (-0.41%)	⬇️

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.

[hardillb]

I had to check in a new package-lock.json to get npm ci to run in the , so something is off, but I've merged main and updated package-lock.json to match

[knolleary]

Still need to double check, but with the new product tiers it is possible for a customer to buy a license for both hub and edge to unlock the full feature set. So we need to handle such license types.

I propose we allow the type to be a , separated list of tier types. This will require a slightly different approach for enabling features - a feature in hub & edge shouldn't get enabled twice because the license enables both.

I suggest we have a codified version of the tickbox table that maps feature to tiers (or tiers to features... not such which way around makes the most sense...)

const tierFeature = {
 featureName: { hub: true, edge: true, enterprise: true },
 ...
}

And then a utility function isFeatureEnabledForTier(string) => boolean - which either handles the comma array of tier types, or gets called multiple times and the result aggregated by the caller.

[hardillb]

Can't that just be the enterprise license that we need to keep to ensure we test everything?

There is also the stuff about maybe including the cert nodes tokens in the license and the new feature listed about access to different BluePrint libraries (that we have not done any planing/design for yet)

[knolleary]

I don't think it can be. We'd want the platform to be able to report what tier they are licensed for - using enterprise as a catch-all will mask what they have actually licensed.

[hardillb]

OK, so we'll need to start again here, multi or mixed tier licenses was not in the original spec issue.

[knolleary]

Need to move bom to hub only.

I do think it would be better to have a map of tier to available features that can be more readily compared to the pricing table to spot any inconsistencies. As it is, we need to step through the code to work out what features have been enabled for each tier and risks things slipping through.

I will do a deeper review to check for any other updates needed.

[knolleary]

BoM has been moved to hub only

[hardillb]

OK, BOM isn't even in the list I was working from

https://github.com/FlowFuse/engineering/issues/230