Skip to content

[31/36] Add OC-150 runtime assertion variable task - #66

Open
cjohnsto-nz wants to merge 2 commits into
feature/runtime-script-variable-evaluationfrom
supervisor/add-runtime-assertion-variable-task
Open

[31/36] Add OC-150 runtime assertion variable task#66
cjohnsto-nz wants to merge 2 commits into
feature/runtime-script-variable-evaluationfrom
supervisor/add-runtime-assertion-variable-task

Conversation

@cjohnsto-nz

@cjohnsto-nz cjohnsto-nz commented Jun 15, 2026

Copy link
Copy Markdown
Owner

Stack PR 31/36. This is the docs-only supervisor handoff that defines OC-150; runtime/editor implementation is intentionally owned by the immediate child PR #67.

This branch adds and indexes:

  • tasks/15-runtime-assertion-variables.md;
  • the OC-150 goal prompt;
  • task-board and dependency-map entries;
  • acceptance criteria and the focused verification matrix.

The task explicitly requires assertion values to be interpolated as runtime data, deterministic unresolved-variable diagnostics, protocol-neutral HTTP/GraphQL/WebSocket/gRPC behavior, stable WebSocket layout, and reuse of the existing variable-aware authoring path. It also states that arbitrary JavaScript source must not use raw script interpolation semantics, consistent with the PR #65 security correction.

No product code belongs in this planning branch. PR #67 carries the implementation and verification evidence.

Stack integrity:

@APKiwi

APKiwi commented Jul 21, 2026

Copy link
Copy Markdown
Collaborator

Verdict: approve. Wired consistently into every index (progress table, dependency map, README, goal prompts) and the acceptance criterion's "detected before execution or reported through deterministic assertion diagnostics" wording correctly anticipates the deferred-to-runtime approach the implementation actually took. The suggested-tests table maps cleanly onto the tests that landed.

@cjohnsto-nz

Copy link
Copy Markdown
Owner Author

Confirmed. No code change is required on this branch.

PR #66 is deliberately the OC-150 task-definition and handoff branch. It adds the task page, goal prompt, task-board/dependency entries, acceptance criteria, and test matrix; the immediate child PR #67 owns the editor/runtime implementation. The task's requirement to resolve assertion templates as data—and its explicit warning against raw JavaScript source interpolation—also aligns with the security correction now landed on PR #65.

I rechecked the live stack after updating PR #65: PR #65 -> PR #66 and PR #66 -> PR #67 are both MERGEABLE/CLEAN, and all existing build/security checks pass. The planning branch therefore remains unchanged.

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.

3 participants