Skip to content

fix: complete model pin and private checkout setup for #191 - #196

Open
jiale-wangOwO wants to merge 1 commit into
LinxISA:codex/aslref-model-pinsfrom
jiale-wangOwO:codex/pr191-model-chain-fix
Open

jiale-wangOwO wants to merge 1 commit into
LinxISA:codex/aslref-model-pinsfrom
jiale-wangOwO:codex/pr191-model-chain-fix

Conversation

@jiale-wangOwO

Copy link
Copy Markdown
Contributor

This is the consolidated dependent fix for #191.

It addresses both observed failures without weakening any guard:

  1. Pin tools/model to isa: regenerate codec for PTO 0.58.5 linx-model#19 (e9f14d9228732f0589f309285cbc8aabaa698353) and update component-lock.v0.58.json atomically.
  2. Make the model job run the codec freshness tests against the immutable v0.58.5 LinxISA authority (660c870ff241f9803d8060f00ed3684ffed6f06c) instead of the superproject's projection, which is intentionally not the codec authority.
  3. Make both workflows authenticate the private tools/SuperScalarModel submodule with the least-privilege SUPERSCALARMODEL_READ_TOKEN secret and document the required repository configuration.

The submodule URL, functional-model lock identity, push-only execution policy, and fail-closed behavior are unchanged. This PR does not add a fallback or bypass a test. Configure SUPERSCALARMODEL_READ_TOKEN in the base repository before expecting the guards job to initialize the private consumer.

Base: #191
Related superseded dependent PRs: #194 and #195.

@jiale-wangOwO
jiale-wangOwO force-pushed the codex/pr191-model-chain-fix branch from 1ca0e23 to 72f316a Compare September 2, 2026 15:34
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