eval: guard shared workspace render ownership - #51
Merged
myobie merged 1 commit intoJul 31, 2026
Conversation
agent-session-id: a078daee-6f98-4916-91a8-d21291407789 agent-tool: Codex CLI agent-tool-version: 0.145.0 agent-model: unknown agent-runtime-profile: /nix/store/mnx8agbdq3wiyb6vz63lhgscgazkrn98-coding-agent-runtime-profile/share/coding-agents/profile.json agent-skills-manifest: /nix/store/5r69m9k2llmri3na81518zx0a7y0d3cn-agent-skills-corpus/share/agent-skills/manifest.json tooling-profile: dotfiles@0fb7e03
myobie
approved these changes
Jul 31, 2026
myobie
left a comment
Collaborator
There was a problem hiding this comment.
Owner review GREEN: focused model-free cell 7/7 on current merged st2 c6846f6; clean merge onto evals main ffe2b14; complete repository-pinned 9887b28 model-free preflight, generated catalog, harness exclusions, fixture resets, no-PII, shell syntax, and diff check all pass. No provider call.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Paired with compoundingtech/st2#95 and merged runtime PR compoundingtech/st2#97.
Discriminator
Adds one model-free cell for two active declarations that share a workspace:
.st2/PERSONA.mdmust fail validationThe cell starts no PTY, Claude, or Codex session. It also updates the generated catalog, explicit model-free harness exclusion, and eval VRS mapping.
AGENT-SPEC.mdremains pinned to its current accepted st2 commit in this PR; the new behavior should become normative there only when the repository-wide pin advances.Current proof
Exact PR head:
1a73f98816cf2c48610bcef59f9aa0de80b8a35d.Against current merged st2
c6846f6239329f0803142afc06c15a07b93937c1(binary SHA2562bba8d58be24250bc262f75f835ce2d780369add275774f3f2135c623d23d29c):The exact head merges cleanly onto current evals main
ffe2b14c785018cf365ea4ad87ef634029732b87. On that temporary merge, the complete repository-pinned model-free preflight passed with exact st29887b2842222def0838c2cd82e6c24c218f7efa6/ binary SHA256d49d44fd4f3f6f655455c212353a469fefa956082bedf22163deb767d8a36a0d:CATALOG.mdchecks passgit diff --checkpassesNo model/provider call was made.
Historical proof
Before runtime PR #97 merged, the original branch proof was also 7/7 against its then-exact runtime head
d691d71. That result is historical; the current proof above supersedes it.