Skip to content

chore(skills): point remaining skill docs at org/public/core#4124

Merged
viktormarinho merged 1 commit into
mainfrom
viktormarinho/skill-docs-org-public
Jun 25, 2026
Merged

chore(skills): point remaining skill docs at org/public/core#4124
viktormarinho merged 1 commit into
mainfrom
viktormarinho/skill-docs-org-public

Conversation

@viktormarinho

@viktormarinho viktormarinho commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

What is this contribution about?

Core skills are now synced into the org/public/core mount (see public-sets.ts), which replaces the image-baked /mnt/skills/public for skill discovery. main already migrated the file-reading and slides docs; this finishes the job for the remaining agent-facing references in xlsx, pdf, docx, pptx and the slides-create comment. The Dockerfile bake and the _bin bare commands (slides-create, pptx-*) still run from /mnt/skills and are intentionally left as-is — retiring those is a separate executable-distribution change.

How to Test

  1. grep -rn "/mnt/skills" packages/sandbox/image/skills/*/SKILL.md → no matches.
  2. The only remaining /mnt/skills refs are the Dockerfile, _bin/* wrappers, and the skills-features.md tracking doc (all load-bearing / accurate).

Migration Notes

Docs-only change; takes effect for agents on the next sandbox image rebuild / skill-set sync. No DB migration, no behavior change.

Review Checklist

  • PR title is clear and descriptive
  • Changes are tested and working
  • Documentation is updated (if needed)
  • No breaking changes

Summary by cubic

Update skill docs and inline comments to use the org/public/core mount instead of /mnt/skills/public for docx, pdf, pptx, xlsx, and slides-create. Docs-only; no behavior changes—remaining /mnt/skills refs in the Dockerfile and _bin wrappers are intentional.

Written for commit 9ada835. Summary will update on new commits.

Review in cubic

Core skills are synced into the org/public/core mount (public-sets.ts),
which replaces the image-baked /mnt/skills/public for discovery — main
already migrated file-reading and slides; this finishes the docs for
xlsx/pdf/docx/pptx and the slides-create comment. The Dockerfile bake and
_bin bare commands (slides-create, pptx-*) still run from /mnt/skills and
are intentionally left as-is — retiring those is a separate executable-
distribution change.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@viktormarinho viktormarinho merged commit a0e3b78 into main Jun 25, 2026
14 checks passed
@viktormarinho viktormarinho deleted the viktormarinho/skill-docs-org-public branch June 25, 2026 18:02
decocms Bot pushed a commit that referenced this pull request Jun 25, 2026
PR: #4124 chore(skills): point remaining skill docs at org/public/core
Bump type: patch

- @decocms/sandbox (packages/sandbox/package.json): 1.4.1 -> 1.4.2

Deploy-Scope: both
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