feat(skills): rewrite oo::self references to published package name

[BlackHole1]

Skill authors can now use backtick-wrapped oo::self::<skill> in their SKILL.md during development. On install, these are resolved to the actual package name (e.g. oo::@oomol/text-tools::<skill>), so cross-skill references remain correct after publication.

Only backtick-delimited occurrences are rewritten; prose mentions of oo::self are left untouched.

[coderabbitai]

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 054062b2-f743-4fc5-9de5-cba430e696a0

📥 Commits

Reviewing files that changed from the base of the PR and between f616dfd and a54b12e.

📒 Files selected for processing (3)

• src/application/commands/skills/index.test.ts
• src/application/commands/skills/registry-skill-markdown.test.ts
• src/application/commands/skills/registry-skill-markdown.ts

---

Summary by CodeRabbit

• Tests

  • Added comprehensive test case and updated test fixtures to validate proper handling of skill documentation during registry operations.

• Bug Fixes

  • Enhanced the skill installation process to correctly resolve internal code references within skill documentation, improving accuracy when installing skills from the registry.

Walkthrough

This PR updates the skill markdown normalization logic to preprocess and resolve oo::self:: backticked references within skill documentation. Changes include a new helper function resolveOoSelfReferences() that replaces `oo::self:: tokens with `oo::${packageName}:: before frontmatter processing, along with corresponding test cases validating the rewriting behavior and updated test fixtures reflecting the new multi-line markdown content structure.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• feat(skills): add published skill installation from registry #41: Directly modifies the same registry-skill-markdown logic that introduces the normalizeInstalledRegistrySkillMarkdown function.
• feat(skills): add update command to check and refresh skills #47: Invokes the markdown-rewriting behavior of normalizeInstalledRegistrySkillMarkdown when preparing and publishing registry skills.

Suggested labels

codex

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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