Skip to content

Harden guardrail routing and imported pricing - #8

Merged
gcjordi merged 2 commits into
mainfrom
fix/guardrail-import-review
Sep 22, 2026
Merged

gcjordi merged 2 commits into
mainfrom
fix/guardrail-import-review

Conversation

@gcjordi

@gcjordi gcjordi commented Sep 22, 2026

Copy link
Copy Markdown
Owner

Summary

  • Make branching Guardrails fail closed unless an exact route is configured.
  • Flag missing or invalid Guardrail routes in the Architecture Analyzer.
  • Preserve legitimate imported pricing only after explicit user confirmation.
  • Preserve trusted locally stored per-node price overrides.
  • Add regression coverage for Guardrail routing, imported pricing, and three-level nested loops with Merge.

Validation

  • 34 core tests pass locally.
  • JavaScript syntax checks pass for all modified runtime files.
  • No new dependency, backend, API key, secret, or external data transmission.
  • Catalan, Spanish, and English are included for the new confirmation text.
  • Full browser test suite could not be executed in the current environment because its browser binary is unavailable.

Scope

The change is confined to the analyzer, importer, model validation, simulator, local storage validation, and regression tests. No schema version or external dependency changes.

Require an explicit route for branching guardrails and flag invalid routing in analysis. Ask for confirmation before importing editable catalog or per-node prices; preserve locally saved overrides. Core tests: 34 passed locally. Browser tests pending; do not merge until validation is complete.

Signed-off-by: Jordi Garcia Castillon <gcjordi@hotmail.com>
Signed-off-by: Jordi Garcia Castillon <gcjordi@hotmail.com>
@gcjordi
gcjordi merged commit 5adf7a1 into main Sep 22, 2026
4 checks passed
@gcjordi
gcjordi deleted the fix/guardrail-import-review branch September 22, 2026 06:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant