Skip to content

fix: replace hardcoded brand placeholders in geo-writing skill#31432

Merged
alex-nork merged 1 commit into
alex-nork/content-automation-geo-skillfrom
run-plan/geo-skill/review-fixes
May 21, 2026
Merged

fix: replace hardcoded brand placeholders in geo-writing skill#31432
alex-nork merged 1 commit into
alex-nork/content-automation-geo-skillfrom
run-plan/geo-skill/review-fixes

Conversation

@alex-nork
Copy link
Copy Markdown
Contributor

Summary

Part of plan: content-automation-geo-skill.md

Closes JARVIS-899

@alex-nork alex-nork requested a review from awlevin as a code owner May 21, 2026 02:29
@linear
Copy link
Copy Markdown

linear Bot commented May 21, 2026

JARVIS-899

Copy link
Copy Markdown

@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: 49a389db5e

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

## PHASE 5 — OUTPUT

Write the completed article as a markdown file to `[YOUR DELIVERABLES FOLDER]/Articles/<slug>.md`. Use kebab-case. Do not include the year in the slug.
Write the completed article as a markdown file to `Articles/Articles/<slug>.md`. Use kebab-case. Do not include the year in the slug.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Write final article to correct output directory

The new output path Articles/Articles/<slug>.md introduces an extra Articles/ segment that is inconsistent with the rest of the skill (Articles/research/<topic-slug>/) and with the prior behavior. In runs where downstream automation or user expectations look for drafts under Articles/<slug>.md, this will place the finished article in the wrong directory and make it appear missing.

Useful? React with 👍 / 👎.

@alex-nork alex-nork merged commit 4795730 into alex-nork/content-automation-geo-skill May 21, 2026
13 checks passed
@alex-nork alex-nork deleted the run-plan/geo-skill/review-fixes branch May 21, 2026 02:34
alex-nork added a commit that referenced this pull request May 21, 2026
…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)
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