Skip to content

Slim Display skill with conditional procedure references - #11

Merged
carlrannaberg merged 1 commit into
mainfrom
codex/slim-display-procedures
Oct 21, 2026
Merged

carlrannaberg merged 1 commit into
mainfrom
codex/slim-display-procedures

Conversation

@carlrannaberg

@carlrannaberg carlrannaberg commented Oct 21, 2026 •

Copy link
Copy Markdown
Contributor

Ordinary publishing currently loads the full email-domain and staged-upload procedures. Move those two conditional procedures into sibling references, reducing the installed entrypoint from 525 lines / 23,924 bytes to 432 lines / 19,219 bytes (19.7%). Keep shared trust boundaries, domain authority rules, upload prerequisites and fallback choices in the entrypoint. Name, description, helpers, comments, and Visualize remain unchanged.

Preserve standalone delivery with one small generator used by CI and release: keep frontmatter first, add an appended-reference notice, and append the same authored files in sorted path order. Missing, unrouted, linked or unsafe references fail generation; no-reference input remains byte-identical. Candidate archives include optional references, all existing mounts are regenerated, and release checks run before publication. This follows the merged consumer preparation in display-dev/display-dev#379.

Validation:

  • 11 archive/standalone checks; helper trust suite; Claude hook suite; structural skill validator; ShellCheck; mount and pinned Visualize parity.
  • Ubuntu 24.04 with repository mounted read-only and networking disabled; GNU release archives built twice with identical bytes and exact extracted references.
  • Seven fresh, uncoached native-agent trials using isolated, network-disabled fixture containers. Raw tool calls and results checked: ordinary publish/sign-in avoid references; listing loads only domain guidance; standalone transfer uses appended guidance; add-on and unavailable-upload requests stop at their boundaries; staged-procedure explanation loads only upload guidance.
  • Exact plaintext prompt copies and raw tool transcripts retained locally. No follow-up steering. These establish procedure behavior, not automatic skill discovery or end-to-end MCP transfer.
  • Independent review caught an omitted listing route; fixed and rechecked with no remaining issues.

Earlier trials with fixture/provider/isolation failures are excluded from the behavioral results. Final tests used an explicitly invoked container runner; the local report records those exclusions and cleanup. No agent-test harness or evidence files are shipped in this PR.

Merging main automatically publishes the source release; this PR does not merge or repin the separate Claude plugin.


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.

@carlrannaberg
carlrannaberg merged commit f7edcda into main Oct 21, 2026
1 check passed
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