[codex] Add source pack glossary

[justin808]

Summary

• add docs/source-pack-glossary.md for source-pack distribution and workflow-contract vocabulary
• link the glossary from the README and seam-design docs
• clarify that root CONTEXT.md remains the batch coordination glossary

Validation

• bin/validate

[coderabbitai]

Warning

Review limit reached

@justin808, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 57 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 7427ab3a-1d63-47a9-84ba-07009aba4db6

📥 Commits

Reviewing files that changed from the base of the PR and between e4ab904 and 491ef5b.

📒 Files selected for processing (3)

• README.md
• docs/seam-design.md
• docs/source-pack-glossary.md

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch jg-codex/document-workflow-glossary

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[claude]

Review summary

This is a clean, docs-only PR that adds docs/source-pack-glossary.md and links it from README.md and docs/seam-design.md. Overall assessment:

• Portability: No consumer-repo-specific commands, labels, branches, or paths are hardcoded — the glossary only defines terms about the source pack itself (Source Pack, Consumer Repo, Agent Workflow Configuration Seam, etc.), consistent with the AGENTS.md "Editing Rules".
• Consistency with existing usage: Cross-checked the new terms against where they're already used in the repo (skills/pr-batch/SKILL.md, skills/plan-pr-batch/SKILL.md, skills/spec/SKILL.md "Canonical Readiness Vocabulary"; docs/review-finding-schema.md; docs/solutions/README.md "Workflow Lessons Library"; README.md "Host Installer Path" / "Codex Native Plugin Path"). The glossary definitions line up with these existing usages.
• Links: All new relative links (docs/source-pack-glossary.md from README, source-pack-glossary.md from docs/seam-design.md, ../CONTEXT.md from the glossary) resolve correctly, and there's no duplicate CONTEXT.md row added to the README table.
• No shell/Ruby helpers changed in this PR, so that review dimension doesn't apply here.
• No security or performance implications — pure documentation change.

One documentation-quality issue flagged inline: the glossary uses an undefined "CE" acronym (lines 72 and 88) that doesn't appear defined anywhere else in the repo, which is ironic for a doc whose purpose is to resolve ambiguous vocabulary.

I did not run bin/validate in this sandbox (it requires elevated approval to execute), but the changes are limited to Markdown content and shouldn't affect any of its checks (frontmatter, Ruby unit tests, installer tests, etc.).

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

[claude]

Review

This is a clean, low-risk documentation-only PR (README + docs/seam-design.md cross-links + new docs/source-pack-glossary.md). No skills, workflows, or bin/ helpers are touched, so the portability/shell-safety focus areas don't really apply here — nothing hardcodes consumer-repo commands, labels, branches, or trackers; every term the glossary defines (Source Pack, Host Installer Path, Native Plugin Path, autoreview, Workflow Lessons Library / docs/solutions/, Review Finding schema) checks out against what's actually in this repo today.

Suggestion (non-blocking): docs/README.md ("ShakaCode Agent Workflow Playbook") has a "Reference Index" table that links out to other cross-cutting reference docs like review-finding-schema.md and coordination-backend.md, but the new source-pack-glossary.md isn't added to that index. Since that page is the stated entry point for finding docs, it'd be easy to miss the glossary unless you already know to check seam-design.md or the README's file-layout table. Consider adding a row there for discoverability.

No correctness, security, or performance issues found. CHANGELOG.md isn't updated for this doc addition, but given the entry style there (mostly feature/behavior changes) that's likely fine to skip — flagging only in case the maintainers want doc-glossary additions logged too.

[claude]

Review

Docs-only change (README.md, docs/seam-design.md, new docs/source-pack-glossary.md) — no skills/, workflows/, or bin/ files touched, so the shared-skill portability rules in AGENTS.md don't apply here.

Checked:

• Both new relative links resolve: ../CONTEXT.md from docs/source-pack-glossary.md, and source-pack-glossary.md from docs/seam-design.md; the README's new links also resolve.
• Glossary terms line up with existing docs — Review Finding matches docs/review-finding-schema.md, Workflow Lessons Library matches docs/solutions/README.md, Host Installer Path matches bin/install-agent-workflows's --host flag, and the "batch coordination" scope carve-out matches CONTEXT.md's actual content.
• No naming collisions: grepped for CE/Compound Engineering and confirmed the new acronym isn't used or defined elsewhere in the repo yet (it's forward-looking prior-art context, consistent with the second commit's intent).
• No security, performance, or shell/Ruby helper concerns — there are no executable changes in this PR.

Nothing blocking. LGTM.