Skip to content

fix(nest-ui): template-literal syntax error + automated frontend validation - #45

Merged
Pentexa merged 2 commits into
mainfrom
fix/nest-ui-runtime
Nov 10, 2026
Merged

Pentexa merged 2 commits into
mainfrom
fix/nest-ui-runtime

Conversation

@Pentexa

@Pentexa Pentexa commented Nov 10, 2026

Copy link
Copy Markdown
Owner

NEST UI runtime fix

Root cause

est/static/index.html line 201 had an extra backtick:
html += ...generated`; — a JavaScript syntax error that prevented the entire inline script from executing, leaving all three panels blank (conversation, world, memory) and no welcome message.

Fix

  • Corrected to ...generated;.
  • Audited the rest of the inline JS (balance checks +
    ode --check): no further defects found.

Guardrails added

est/tools/frontend_audit.py: extracts inline script, runs
ode --check (fallback balance check), optional --smoke.

est/tools/frontend_smoke.node.js: executes the real inline script in a DOM stub with backend-shaped snapshots; asserts welcome message, world canvas draw calls, memory stats/hypothesis, send/Enter flow, and control-button wiring.

  • CI
    est job now runs rontend_audit.py --smoke.

Verification

  • Node syntax check: OK. Balance checks: OK.
  • DOM smoke: 8/8 assertions PASS (welcome, world renders, memory panel, hypothesis list, user/bot send, start wiring).
  • Served page on 127.0.0.1:8765 contains the fixed line and zero double-backticks; browser tab loads clean (title METARGON NEST, no load error).
  • Backend API confirmed: /api/state, /api/config respond; NEST unit suite 10/10 OK (incl. demo tests + server endpoints + frontend validation).
  • Repo hygiene OK; CI YAML valid (jobs: repo-checks, python-v09, academy, generalization-audit, nest, rust).

Files changed

est/static/index.html (1-line fix)

est/tools/frontend_audit.py (new)

est/tools/frontend_smoke.node.js (new)

est/tests/test_nest.py (+2 frontend validation tests)

  • .github/workflows/ci.yml (nest job: frontend validation step)

No Academy learning algorithms, frozen research results, or historical artifacts were modified.

Copilot AI balanced review requested due to automatic review settings November 10, 2026 15:12

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

…lies, pending state); extend DOM smoke coverage
@Pentexa
Pentexa merged commit 058d526 into main Nov 10, 2026
7 checks passed
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.

2 participants