Skip to content

Docs/Skills: remove hidden .codex skill; ship v3 skill pack under skills/#88

Merged
AmirTlinov merged 1 commit intomainfrom
issue-87-skillpack-visible-v3
Mar 3, 2026
Merged

Docs/Skills: remove hidden .codex skill; ship v3 skill pack under skills/#88
AmirTlinov merged 1 commit intomainfrom
issue-87-skillpack-visible-v3

Conversation

@AmirTlinov
Copy link
Copy Markdown
Owner

@AmirTlinov AmirTlinov commented Mar 3, 2026

Closes #87.

What changed

  • Removed repo-local hidden skill under .codex/skills/*.
  • Promoted the visible distributable skill pack under skills/explorer-context-pack to be the canonical v3 docs for users cloning this repo.
  • Replaced old LEGACY/v2 reference files with v3-only progressive-disclosure references (00..60).

Why

The skills/ directory is intended for downstream users who download this repository for their agents; .codex/skills/ is unnecessary noise inside the repo.

Summary by CodeRabbit

  • Documentation
    • Updated documentation structure to align with v3 standards and contracts.
    • Removed legacy documentation and deprecation notices.
    • Clarified and reorganized reference guides with explicit v3 requirements, handoff protocols, and daily workflows.
    • Updated reference links for improved navigation.

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 41466d2 and 4ec6d9c.

📒 Files selected for processing (15)
  • .codex/skills/context-pack-repo-profile/SKILL.md
  • skills/explorer-context-pack/SKILL.md
  • skills/explorer-context-pack/references/00-v3-contract-cheatsheet.md
  • skills/explorer-context-pack/references/10-daily-workflows.md
  • skills/explorer-context-pack/references/20-finalize-qa-freshness.md
  • skills/explorer-context-pack/references/30-recovery-playbook.md
  • skills/explorer-context-pack/references/40-section-templates.md
  • skills/explorer-context-pack/references/50-naming-conventions.md
  • skills/explorer-context-pack/references/60-anti-patterns.md
  • skills/explorer-context-pack/references/actions.md
  • skills/explorer-context-pack/references/examples.md
  • skills/explorer-context-pack/references/qa-dod.md
  • skills/explorer-context-pack/references/recovery.md
  • skills/explorer-context-pack/references/role-modes.md
  • skills/explorer-context-pack/references/troubleshooting.md
💤 Files with no reviewable changes (7)
  • skills/explorer-context-pack/references/recovery.md
  • skills/explorer-context-pack/references/examples.md
  • skills/explorer-context-pack/references/qa-dod.md
  • skills/explorer-context-pack/references/actions.md
  • skills/explorer-context-pack/references/troubleshooting.md
  • .codex/skills/context-pack-repo-profile/SKILL.md
  • skills/explorer-context-pack/references/role-modes.md

📝 Walkthrough

Walkthrough

This PR removes the legacy .codex/skills documentation directory and establishes skills/explorer-context-pack as the canonical v3-only distributable skill pack. It updates reference documentation with versioned headings, removes outdated legacy guidance files, and corrects a relative link path.

Changes

Cohort / File(s) Summary
Removal of legacy repo-local skill
.codex/skills/context-pack-repo-profile/SKILL.md
Deleted entire context-pack v3 profile description, contract, and related guidance documentation.
Migration to v3 distributable skill
skills/explorer-context-pack/SKILL.md
Replaced v2 legacy description with v3 context_pack documentation; added explicit v3 contract details (input/output actions, write behavior, revision requirements, finalization sequence, handoff criteria); introduced 60-second daily path with concrete steps; expanded references section and minimal handoff protocol.
Reference documentation title updates
skills/explorer-context-pack/references/{10-daily-workflows, 20-finalize-qa-freshness, 30-recovery-playbook, 40-section-templates, 50-naming-conventions, 60-anti-patterns}.md
Updated document headings to add "context_pack v3" versioned prefix for consistency and clarity.
Legacy reference files removal
skills/explorer-context-pack/references/{actions, qa-dod, recovery, role-modes, troubleshooting}.md
Deleted legacy v2 contract mappings, deprecated notices, and outdated diagnostic guidance; consolidated this content into current v3 reference materials.
Relative link correction
skills/explorer-context-pack/references/00-v3-contract-cheatsheet.md
Updated relative path to truth source from ../../../../TECHNICAL.md to ../../../TECHNICAL.md.
Legacy notices cleanup
skills/explorer-context-pack/references/{examples}.md
Removed legacy notice blocks and redirects; retained current v3 canonical content.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related issues

Possibly related PRs

Poem

🐰 Legacy files hop away, v3 shines bright today!
Distributable skill takes the stage with refined prose—
Fresh references bloom where old wisdom goes. ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly summarizes the main change: removing the hidden .codex skill and promoting the visible skills/ directory as the canonical v3 distribution.
Linked Issues check ✅ Passed All linked issue requirements are met: .codex/skills files removed, skills/explorer-context-pack promoted as v3-only with updated SKILL.md and progressive-disclosure references replacing legacy content.
Out of Scope Changes check ✅ Passed All changes are directly aligned with issue #87 objectives: removing .codex/skills and upgrading skills/explorer-context-pack documentation to v3-only distribution format.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch issue-87-skillpack-visible-v3

Comment @coderabbitai help to get the list of available commands and usage tips.

@AmirTlinov
Copy link
Copy Markdown
Owner Author

Independent review (docs-vault lens) PASS.

Checked:

  • no tracked .codex/skills/* remains
  • skills/explorer-context-pack is v3-only + references resolve
  • no remaining refs to deleted .codex/skills/context-pack-repo-profile.

@AmirTlinov AmirTlinov merged commit 376a92c into main Mar 3, 2026
2 checks passed
@AmirTlinov AmirTlinov deleted the issue-87-skillpack-visible-v3 branch March 3, 2026 14:06
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.

Docs/Skills: remove repo-local .codex/skills and promote skills/ as distributable v3 skill

1 participant