feat: rename geo -> geo-writing, rewrite bootstrap for skill-first onboarding#31380
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: d0f6a99b27
ℹ️ About Codex in GitHub
Codex has been enabled to automatically 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 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
|
|
||
| ## Edit loop | ||
| ### Reading comments | ||
| When you see open comments in context (via the document_comments injector), read every one. Don't cherry-pick. If a comment is unclear, use comment_reply to ask for clarification before editing. Don't guess. |
There was a problem hiding this comment.
Use only registered tools in comment-driven edit loop
This prompt instructs the assistant to call comment_reply (and later comment_resolve), but no such tools are registered in the codebase—the bundled document toolset only exposes document_create, document_update, document_read, document_list, and document_delete (assistant/src/config/bundled-skills/document/TOOLS.json). In content-automation sessions, following this bootstrap will produce tool-not-found failures exactly when comments are unclear or need resolution, which breaks the primary editing workflow.
Useful? React with 👍 / 👎.
636ca57
into
alex-nork/content-automation-geo-skill
…ARVIS-899) (#31378) * feat: add GEO article writing skill as bundled skill (#31369) * feat: update content-automation bootstrap for GEO article flow (#31371) * fix: move geo skill reference files to references/ subdir, remove orphaned drafts, add compatibility field (#31376) * fix: remove duplicated geo skill rules from bootstrap template (#31377) * feat: rename geo -> geo-writing, rewrite bootstrap for skill-first onboarding (#31380) * feat: rename geo skill to geo-writing, rewrite bootstrap for skill-first onboarding * test: update bootstrap tests for skill-first onboarding rewrite * feat: rewrite wake-up greeting to GEO message for content-automation cohort (#31384) * fix: replace hardcoded brand placeholders with dynamic references in geo-writing skill (#31432)
Summary
geotogeo-writing(directory + frontmatter name)Part of plan: content-automation-geo-skill.md
Closes JARVIS-899