Skip to content

Fix settings validation asset drift#137

Merged
Pigbibi merged 1 commit into
mainfrom
codex/fix-settings-drift-20260701
Jul 1, 2026
Merged

Fix settings validation asset drift#137
Pigbibi merged 1 commit into
mainfrom
codex/fix-settings-drift-20260701

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Summary

  • sync internal dependency matrix with current consumer repository pins
  • regenerate strategy switch page/profile assets from origin/main and include app_css/app_js in the sync script
  • make Validate check bundled app assets so deploy-time JS cannot drift from source again
  • include IBIT Z-Score exit changes in pending-change detection and summary rows
  • update worker validation tests for split HTML/CSS/JS assets and current cash-only defaults

Tests

  • python3 python/scripts/build_config.py --check
  • python3 python/scripts/runtime_settings.py validate
  • python3 python/scripts/check_internal_dependency_matrix.py --projects-root /home/ubuntu/Projects --json --strict --require-consumer-files
  • python3 -m unittest discover -s python/tests -v
  • node --experimental-default-type=module tests/strategy_switch_worker_validation.mjs
  • node --check for index script, page/profile/app CSS/app JS assets, and worker.js
  • git diff --check

@github-actions

github-actions Bot commented Jul 1, 2026

Copy link
Copy Markdown

🤖 Codex PR Review

⚠️ Review skipped: The Codex review could not be completed.

No Codex service URL or API key configured. Set CODEX_AUDIT_SERVICE_URL, ANTHROPIC_API_KEY, or OPENAI_API_KEY.

Please ensure a human reviewer checks this PR before merging.

@Pigbibi

Pigbibi commented Jul 1, 2026

Copy link
Copy Markdown
Contributor Author

CI note: the current python failure is an ordering dependency on QuantStrategyLab/UsEquitySnapshotPipelines#149. This PR updates internal_dependency_matrix.json to expect UsEquityStrategies@c8df5f9659340965bd7f53998892ed1018ed4254, while the Validate job checks out consumer repos from their main branches; UsEquitySnapshotPipelines main still pins 2d092fc until #149 is merged. Local strict matrix validation passes against the workspace with #149 applied.

@Pigbibi Pigbibi merged commit a65163e into main Jul 1, 2026
8 of 10 checks passed
@Pigbibi Pigbibi deleted the codex/fix-settings-drift-20260701 branch July 1, 2026 06:18
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