Repository navigation
Vendor website-builder-journal README and normalize skill pin to 38c6433 - #85
Conversation
Copy website-builder-journal from skill-lib main 38c64332b840b2bbe1c07e53aeee8996644548e9 with propagate_skills.py, adding the canonical README.md that was missing. SKILL.md is byte-identical between 0981aed and 38c6433. Rewrite .agents/skills/README.md in the standard backticked "Source commit:" format, keeping the local usage line. Update the pin in README.md and tests/site-contract.test.mjs. Append the required By the builder journal entry.
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7b00a03199
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| "id": "2026-10-06-a-copy-that-says-where-it-came-from", | ||
| "date": "2026-10-06", | ||
| "time": "03:56:36-07:00", | ||
| "model": "Grok Bot; exact runtime model withheld", |
There was a problem hiding this comment.
For this non-journal website transaction, the new record explicitly says the exact model was withheld and substitutes the agent label Grok Bot. The vendored contract in .agents/skills/website-builder-journal/SKILL.md requires the full executing model, says product/agent labels are insufficient, and requires the modification to remain blocked when that identity is unavailable. The current gate accepts this composite label, so the passing tests do not make the entry compliant; merging would also publish it behind the journal's immutable append-only boundary.
Useful? React with 👍 / 👎.
1 participant
Erin merges. This PR is just the sync. Nothing here merges it.
Source
38c64332b840b2bbe1c07e53aeee8996644548e9(38c6433)0981aed7695ba2675d5de35ef43ba734e94adea0. New pin:38c64332b840b2bbe1c07e53aeee8996644548e9Skills updated
website-builder-journal, viatools/propagate_skills.py --apply:README.md, which was missing. This is the gap the skill-lib consumer-drift check flags.SKILL.mdis unchanged: canonicalwebsite-builder-journal/is byte-identical between0981aedand38c6433.Pin format normalized
.agents/skills/README.mdnow uses the standardSource commit: `<sha>`line thatpropagate_skills.py/check_consumer_drift.pyread, instead of the old- commit:bullet list. I kept the local "Usage: load … before any website write" line.README.md("Maintaining the builder journal") andtests/site-contract.test.mjsmove their pin from0981aed…to38c6433…to match.Builder journal
src/_data/builder.json, aswebsite-builder-journalrequires. The 7 prior records are unchanged.modelisGrok Bot; exact runtime model withheld. This follows the precedent of the two prior entries and your instruction there to state what the session exposes and note what it withholds.hmmm"), this attribution is your call at merge.CI and regenerated files
npm run check:llmspasses unchanged.npm run check:builder -- --base origin/main: preserved 7, appended 1, passtests/site-contract.test.mjs: 13/13 passnpm run checkneeds the CIGITHUB_TOKENfor live textbook refresh, so the PR workflow is the verification of record.