Skip to content

docs: add tokenizer and state contract Codex skills - #33

Merged
cgfixit merged 1 commit into
mainfrom
codex/insight-verification-skills
Sep 6, 2026
Merged

docs: add tokenizer and state contract Codex skills#33
cgfixit merged 1 commit into
mainfrom
codex/insight-verification-skills

Conversation

@cgfixit

@cgfixit cgfixit commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Proposed changes

Add insight-tokenizer-check and insight-state-contract as self-contained repository
skills with Codex selector metadata. Update the Codex dispatch map with invocation guidance.

  • Tokenizer checks bound failure reproductions and cover invalid budgets, overlap,
    zero-overlap tail coverage, and lazy loading.
  • State checks cover malformed schema, partial mutation, legacy compatibility,
    path semantics, and lazy restoration after model-name changes.

Invariant/governance impact: instruction-only; diagnose by default and implement fixes
when requested. Runtime code, dependency manifests, tests, CI gates, and output schemas
are unchanged. Personal Codex copies were also installed locally and hash-verified;
machine-specific paths are not part of these skills.

Types of changes

  • Codex skills / documentation
  • Runtime behavior
  • Dependencies

Benefits / why

Turn the setup review's two suggested workflows into invocable skills grounded in the
current source. Future tokenizer regressions can be reproduced without hanging a test
runner, and state checks explicitly detect partially applied loads.

Risks to monitor

The skills describe checks, not runtime fixes. Existing tokenizer and state issues
remain. Personal and repository copies should be synchronized; same-name skills are
not merged. A new task/restart may be needed to refresh the skill catalog.

Completed checklist

  • Both repository and personal skill metadata validate
  • Personal copies and UI metadata match repository hashes
  • Ruff lint and formatting pass
  • Strict mypy passes (10 source files)
  • Offline unit tests: 118 passed on Windows Python 3.12.0rc3
  • Diff whitespace and named-file staging reviewed
  • Real-model integration / Python 3.13: skipped for instruction-only changes
  • Runtime bugs fixed: outside this skill-authoring scope

Further comments

Invoke $insight-tokenizer-check or $insight-state-contract, or use the skill selector.
No new tests or runtime helpers were needed for these instruction-only additions.

Branch naming / merge order / base commit

Branch: codex/insight-verification-skills. Target: main.
Follows merged setup PR #32; no unmerged dependency.
Base commit: 1128c0a5ae6918479de6ed3f156ef6459f925bd4.

@cgfixit
cgfixit marked this pull request as ready for review September 6, 2026 15:05
@cgfixit
cgfixit merged commit 8b20871 into main Sep 6, 2026
8 checks passed
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 6, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-06T15:07:10.985381Z ef64f58 Draft marked ready
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

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