Reorganize AI artifacts under ai-artifacts - #20
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 89badb8b3e
ℹ️ 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".
There was a problem hiding this comment.
Pull request overview
This PR reorganizes AI-related repo assets under the new ai-artifacts/ root, retargets documentation + sync tooling to the new paths, and introduces additional “source of truth” artifacts (skills, instructions, READMEs) to support multi-harness workflows.
Changes:
- Move/establish AI artifact families under
ai-artifacts/(skills, instructions, hooks, MCP config, prompts, plugins, agents, output styles). - Update sync tooling (
scripts/sync-skills.ps1) and documentation to referenceai-artifacts/skills/shared/as the skill source-of-truth. - Add/refresh core context + operational docs (new root
CONTEXT.md, updates acrossAGENTS.md, harness docs, and scratch PRDs).
Reviewed changes
Copilot reviewed 47 out of 117 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| VSCode_Extension/adoc-preview-menu-1.0.0/package.json | Adds VS Code extension manifest for AsciiDoc preview context menu |
| VSCode_Extension/adoc-preview-menu-1.0.0/extension.js | Adds VS Code extension command registrations |
| THIRD-PARTY/README.md | Removes third-party notices document |
| THIRD-PARTY/mattpocock-skills.LICENSE | Removes vendored upstream license copy |
| THIRD-PARTY/bradautomates-claude-video.LICENSE | Removes vendored upstream license copy |
| scripts/sync-skills.ps1 | Retargets skill source root + warnings to ai-artifacts/skills/shared/ |
| instructions/README.md | Removes old instructions README (replaced under ai-artifacts) |
| docs/reviews/2026-07-04-fable-repo-review.md | Updates historical review to new artifact paths |
| docs/okf-adoption.md | Updates OKF adoption doc to new skill metadata paths |
| docs/harnesses/copilot-cli.md | Updates harness doc references to new skills root |
| docs/harnesses/codex.md | Updates Codex harness doc to new artifact-type homes |
| docs/harnesses/claude-code.md | Updates Claude Code harness doc to new artifact paths (skills/output-styles) |
| CONTEXT.md | Adds root glossary/terminology and ambiguity notes for the repo |
| ai-artifacts/skills/shared/setup/setup-pre-commit/SKILL.md | Adds setup skill for pre-commit tooling |
| ai-artifacts/skills/shared/setup/setup-pre-commit/METADATA.md | Adds OKF metadata for setup-pre-commit |
| ai-artifacts/skills/shared/setup/import-upstream-skill/SKILL.md | Updates import-upstream-skill to new source-of-truth paths |
| ai-artifacts/skills/shared/setup/import-upstream-skill/METADATA.md | Adds OKF metadata for import-upstream-skill |
| ai-artifacts/skills/shared/setup/git-guardrails/SKILL.md | Adds skill describing git command guardrail hook setup |
| ai-artifacts/skills/shared/setup/git-guardrails/scripts/block-dangerous-git.sh | Adds bash hook script to block dangerous git commands |
| ai-artifacts/skills/shared/setup/git-guardrails/scripts/block-dangerous-git.ps1 | Adds PowerShell hook script to block dangerous git commands |
| ai-artifacts/skills/shared/setup/git-guardrails/METADATA.md | Adds OKF metadata for git-guardrails (upstream-derived) |
| ai-artifacts/skills/shared/setup/check-skill-updates/SKILL.md | Updates check-skill-updates skill to new skills root path |
| ai-artifacts/skills/shared/setup/check-skill-updates/METADATA.md | Adds OKF metadata for check-skill-updates |
| ai-artifacts/skills/shared/session/write-a-skill/SKILL.md | Adds session skill for authoring skills |
| ai-artifacts/skills/shared/session/write-a-skill/SCRIPTS.md | Adds guidance for bundling scripts with skills |
| ai-artifacts/skills/shared/session/write-a-skill/METADATA.md | Adds OKF metadata for write-a-skill |
| ai-artifacts/skills/shared/session/write-a-skill/EXAMPLES.md | Adds examples for write-a-skill |
| ai-artifacts/skills/shared/session/recon/SKILL.md | Adds recon skill for environment fact-finding via probes |
| ai-artifacts/skills/shared/session/recon/RECIPES.md | Adds per-stack probe “recipes” for recon |
| ai-artifacts/skills/shared/session/recon/METADATA.md | Adds OKF metadata for recon |
| ai-artifacts/skills/shared/session/handoff/SKILL.md | Adds handoff skill definition |
| ai-artifacts/skills/shared/session/handoff/METADATA.md | Adds OKF metadata for handoff |
| ai-artifacts/skills/shared/session/grill-me/SKILL.md | Adds grill-me skill definition with optional docs-mode |
| ai-artifacts/skills/shared/session/grill-me/METADATA.md | Adds OKF metadata for grill-me |
| ai-artifacts/skills/shared/session/grill-me/CONTEXT-FORMAT.md | Adds CONTEXT.md formatting guidance used by docs-mode |
| ai-artifacts/skills/shared/session/grill-me/ADR-FORMAT.md | Adds ADR formatting guidance used by docs-mode |
| ai-artifacts/skills/shared/session/caveman/SKILL.md | Adds compressed-response “caveman” mode skill |
| ai-artifacts/skills/shared/session/caveman/METADATA.md | Adds OKF metadata for caveman |
| ai-artifacts/skills/shared/README.md | Updates skills origin map/readme to new paths |
| ai-artifacts/skills/shared/planning/scratch/SKILL.md | Adds scratch skill for backlog capture/listing |
| ai-artifacts/skills/shared/planning/scratch/RANKING.md | Adds scratch ranking formula + tiebreakers |
| ai-artifacts/skills/shared/planning/scratch/METADATA.md | Adds OKF metadata for scratch |
| ai-artifacts/skills/shared/planning/scratch/LAYOUT.md | Adds canonical .scratch/ layout definition |
| ai-artifacts/skills/shared/planning/scratch-plan/SKILL.md | Adds scratch-plan skill for ranking backlog items |
| ai-artifacts/skills/shared/planning/scratch-plan/METADATA.md | Adds OKF metadata for scratch-plan |
| ai-artifacts/skills/shared/documents/mail-to-adoc/tests/test_mail_to_adoc.py | Adds regression tests for mail-to-adoc tooling |
| ai-artifacts/skills/shared/documents/mail-to-adoc/tests/README.md | Adds test README (currently with stale paths) |
| ai-artifacts/skills/shared/documents/mail-to-adoc/tests/conftest.py | Adds pytest fixtures for mail-to-adoc tests |
| ai-artifacts/skills/shared/documents/mail-to-adoc/SKILL.md | Adds mail-to-adoc skill definition (currently with stale paths) |
| ai-artifacts/skills/shared/documents/mail-to-adoc/scripts/Move-Belege.ps1 | Adds PowerShell helper script for file moves/dedup |
| ai-artifacts/skills/shared/documents/mail-to-adoc/README.md | Adds mail-to-adoc README (currently with stale paths) |
| ai-artifacts/skills/shared/documents/mail-to-adoc/METADATA.md | Adds OKF metadata for mail-to-adoc |
| ai-artifacts/skills/shared/documents/mail-to-adoc/docs/prd/PRD.md | Adds PRD for mail-to-adoc converter behavior |
| ai-artifacts/skills/shared/documents/mail-to-adoc/docs/prd/PRD_EDGE_CASES.md | Adds edge-case PRD addendum/test matrix |
| ai-artifacts/skills/shared/documents/mail-to-adoc/attachment-blocklist.txt | Adds attachment blocklist (currently contains NUL bytes) |
| ai-artifacts/skills/shared/coding/zoom-out/SKILL.md | Adds zoom-out skill |
| ai-artifacts/skills/shared/coding/zoom-out/METADATA.md | Adds OKF metadata for zoom-out |
| ai-artifacts/skills/shared/coding/tdd/test-doubles.md | Adds TDD reference: test double selection |
| ai-artifacts/skills/shared/coding/tdd/stacks/sql.md | Adds stack rules for SQL TDD |
| ai-artifacts/skills/shared/coding/tdd/stacks/python.md | Adds stack rules for Python TDD |
| ai-artifacts/skills/shared/coding/tdd/stacks/powershell.md | Adds stack rules for PowerShell TDD |
| ai-artifacts/skills/shared/coding/tdd/stacks/csharp.md | Adds stack rules for C# TDD |
| ai-artifacts/skills/shared/coding/tdd/SKILL.md | Adds TDD workflow skill |
| ai-artifacts/skills/shared/coding/tdd/reviewing-and-cycle.md | Adds reference: cycle discipline + reviewing tests |
| ai-artifacts/skills/shared/coding/tdd/METADATA.md | Adds OKF metadata for tdd |
| ai-artifacts/skills/shared/coding/tdd/design-for-testability.md | Adds reference: design for testability |
| ai-artifacts/skills/shared/coding/tdd/companions.md | Adds reference: companion methodologies |
| ai-artifacts/skills/shared/coding/tdd/behaviors.md | Adds reference: behaviors + ambiguity handling |
| ai-artifacts/skills/shared/coding/prototype/SKILL.md | Adds prototype workflow skill |
| ai-artifacts/skills/shared/coding/prototype/METADATA.md | Adds OKF metadata for prototype |
| ai-artifacts/skills/shared/coding/improve-codebase-architecture/SKILL.md | Adds architecture improvement skill |
| ai-artifacts/skills/shared/coding/improve-codebase-architecture/METADATA.md | Adds OKF metadata for improve-codebase-architecture |
| ai-artifacts/skills/shared/coding/improve-codebase-architecture/LANGUAGE.md | Adds architecture vocabulary glossary |
| ai-artifacts/skills/shared/coding/improve-codebase-architecture/INTERFACE-DESIGN.md | Adds reference for interface design iteration |
| ai-artifacts/skills/shared/coding/improve-codebase-architecture/HTML-REPORT.md | Adds reference for HTML report scaffold |
| ai-artifacts/skills/shared/coding/improve-codebase-architecture/DEEPENING.md | Adds reference for safe deepening strategy |
| ai-artifacts/skills/shared/coding/diagnose/SKILL.md | Adds diagnose skill (debugging discipline) |
| ai-artifacts/skills/shared/coding/diagnose/scripts/hitl-loop.template.sh | Adds bash HITL loop template |
| ai-artifacts/skills/shared/coding/diagnose/scripts/hitl-loop.template.ps1 | Adds PowerShell HITL loop template |
| ai-artifacts/skills/shared/coding/diagnose/METADATA.md | Adds OKF metadata for diagnose |
| ai-artifacts/prompts/README.md | Adds README for prompts artifact family |
| ai-artifacts/plugins/README.md | Adds plugins README (currently missing ai-artifacts/ prefix in paths) |
| ai-artifacts/output-styles/README.md | Adds README for output-styles artifact family |
| ai-artifacts/mcp-config/README.md | Adds README for MCP config artifact family |
| ai-artifacts/instructions/README.md | Adds new instructions README under ai-artifacts |
| ai-artifacts/instructions/anthropic/claude-ai/profile.md | Adds repo copy of claude.ai profile instructions |
| ai-artifacts/hooks/README.md | Adds README for hooks artifact family |
| ai-artifacts/agents/README.md | Adds README for agents artifact family |
| AGENTS.md | Updates repo operational guide to new artifact roots and paths |
| .scratch/understand-scratch-skill/PRD.md | Updates scratch PRD references to new skill paths |
| .scratch/sync-skills-drift-check/PRD.md | Updates drift-check PRD references to new skill paths |
| .scratch/repo-scope-strays/PRD.md | Updates scratch PRD wording/path references |
| .scratch/repo-scaffold/PRD.md | Updates scaffold PRD to new artifact roots |
| .scratch/repo-scaffold/artifacts/HANDOFF-ai-lab-repo-structure.md | Updates scaffold handoff artifact to reflect ai-artifacts tree |
| .scratch/public-repo-compliance/PRD.md | Updates compliance PRD (but now conflicts with removed THIRD-PARTY files) |
| .scratch/profile-improvement/PRD.md | Updates profile improvement PRD to new profile path |
| .scratch/mail-to-doc/PRD.md | Updates mail-to-doc scratch PRD references to new skill path |
| .scratch/mail-to-doc/issues/02-analyse-effectiveness.md | Updates issue references to new skill paths |
| .scratch/mail-to-doc/issues/01-incorporate-latest-changes.md | Updates issue references to new skill paths |
| .scratch/incorporate-global-claude-setup/PRD.md | Updates claude setup PRD references to new profile path |
| .scratch/import-upstream-skills/PRD.md | Updates import PRD references to new skill paths |
| .scratch/import-upstream-skills/issues/05-import-setup-hook-skills.md | Updates issue progress entries to new skill paths |
| .scratch/import-upstream-skills/issues/03-import-coding-skills.md | Updates issue progress entries to new skill paths |
| .scratch/fetch-latest-claude-skills/PRD.md | Updates PRD references to new import skill path |
| .scratch/docs-integrity-pass/PRD.md | Updates integrity-pass PRD references to new paths |
| .scratch/claude-md-planning-defaults/PRD.md | Updates PRD references (but retains an old path mention) |
| .scratch/claude-code-skill-adaptation/PRD.md | Updates adaptation PRD references to new artifact roots |
| .scratch/claude-code-skill-adaptation/issues/01-audit-and-classify.md | Updates issue acceptance criteria paths |
| .scratch/check-updates-detection-scope/PRD.md | Updates PRD references to new hooks/artifact paths |
| .scratch/AGENTS.md | Updates scratch folder guide to new deliverable homes |
| .scratch/agents-md-folder-guides/PRD.md | Updates PRD references to new artifact roots |
| .scratch/add-watch-skill/artifacts/watch/CHANGELOG.md | Updates changelog reference to new hooks path |
| .gitignore | Expands ignore rules; updates generated-mirror comments to new skills root |
| .editorconfig | Adds EditorConfig defaults (UTF-8, LF, whitespace trimming, indents) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@copilot please see my comment |
- Run markdownlint --fix to auto-fix MD049/MD047/MD022/MD032/MD034/MD029 - Fix MD025 (multiple H1): change # Skill Metadata and # Citations to ## in all METADATA.md files; change # Citations to ## in docs/okf-adoption.md - Fix MD025 in .markdownlint.json: set front_matter_title to empty string so files with YAML frontmatter title + body H1 don't trigger false positives - Fix MD040 (fenced code without language): add 'text' language to 6 code blocks - Fix MD036 (emphasis as heading): convert bold examples to #### headings in session/caveman/SKILL.md - Fix MD060 (table column alignment): reformat tables with proper padding, using display-width-aware alignment for emoji characters
- fix yamllint spacing issue in dependabot workflow - remove disallowed global strict mode usage in workflow bash scripts - fix shellcheck SC2320 in write-summary scripts - apply ruff format to simulate-workflows test file Co-authored-by: Dennis Winter <dennis.winter@example.com> Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Summary
ai-artifacts/CONTEXT.mdand clean up stale path references after the rebaseNotes
origin/main