Skip to content

docs(skills): make skill references repo-relative#68

Merged
davidahmann merged 1 commit intomainfrom
codex/skills-relative-path-cleanup
Feb 26, 2026
Merged

docs(skills): make skill references repo-relative#68
davidahmann merged 1 commit intomainfrom
codex/skills-relative-path-cleanup

Conversation

@davidahmann
Copy link
Contributor

Problem

Several skill docs used machine-specific absolute paths (for example /Users/davidahmann/...), which breaks portability in other workspaces and CI environments.

Changes

  • Replaced absolute repository path references with repo-relative references across skill docs.
  • Updated intra-skill handoff links to use relative skill paths.
  • Normalized path examples in workflow sections to remain executable across environments.

Validation

  • make prepush-full
  • ./.tmp/wrkr scan -path . --json > .tmp/command-anchor-scan.json

@davidahmann davidahmann merged commit 5a6a3e1 into main Feb 26, 2026
7 checks passed
@davidahmann davidahmann deleted the codex/skills-relative-path-cleanup branch February 26, 2026 11:31
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: fd19a48b1a

ℹ️ 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".

- Structure references (match level of detail and style):
- `/Users/davidahmann/Projects/gait/product/PLAN_v1.md`
- Output file: `/Users/davidahmann/Projects/wrkr/product/PLAN_NEXT.md` (unless user specifies a different target)
- `gait/product/PLAN_v1.md` (reference only, if available in your environment)

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Fix relative path to sibling gait style reference

The converted reference path gait/product/PLAN_v1.md resolves under the current repo (wrkr/gait/...), but the original absolute path pointed to a sibling checkout (.../Projects/gait/...), which should become ../gait/product/PLAN_v1.md. In environments where wrkr and gait are side-by-side (the layout implied by the previous path), this change prevents the workflow from finding the style reference and silently drops intended plan-structure guidance.

Useful? React with 👍 / 👎.

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