Skip to content

Added GEO principles to SKILL.md#6497

Merged
JV0812 merged 14 commits into
mainfrom
add-geo-guide
Jun 26, 2026
Merged

Added GEO principles to SKILL.md#6497
JV0812 merged 14 commits into
mainfrom
add-geo-guide

Conversation

@JV0812

@JV0812 JV0812 commented Mar 26, 2026

Copy link
Copy Markdown
Collaborator

Purpose of this pull request

This pull request is to create a SKILL.md file with GEO principles to enhance the AI citation.

Select the type of change

  • Minor Changes - Typos, formatting, slight revisions
  • Update Content - Revisions, updating sections
  • New Content - New features, sections, pages, tutorials
  • Site and Tools - .clabot, version updates, maintenance, dependencies, new packages for the site (Docusaurus, Gatsby, React, etc.)

Ticket (if applicable)

Create SKILL.md for GEO principles

@cla-bot cla-bot Bot added the cla-signed Contributor approved, listed in .clabot file label Mar 26, 2026
@JV0812 JV0812 self-assigned this Mar 26, 2026
@JV0812 JV0812 added the do-not-merge Pull requests that should not be merged label Mar 26, 2026

@mafsumo mafsumo left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@JV0812 great start! I added a comments for review.

Comment thread .claude/skills/geo-guide/SKILL.md Outdated
Comment thread .claude/skills/geo-guide/SKILL.md
Comment thread .claude/skills/geo-guide/SKILL.md Outdated
Comment thread .claude/skills/geo-guide/SKILL.md
@JV0812

JV0812 commented Apr 6, 2026

Copy link
Copy Markdown
Collaborator Author

@mafsumo Thanks for your review and feedback. I have incorporated your comments. Please let me know if any updates needed.

@JV0812 JV0812 requested a review from mafsumo April 6, 2026 13:41
@JV0812 JV0812 requested a review from vfalconisumo as a code owner May 11, 2026 10:53

@kimsauce kimsauce left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The structure is solid — clear trigger guards, good JSON-LD FAQ schema pattern, and the five principles are well-organized. A few issues need fixing before merge.

Critical

1. British spelling throughout

"optimise/Optimisation" appears in the frontmatter name/description fields and throughout the body. Needs to be American English everywhere:

  • OptimisationOptimization
  • optimiseoptimize
  • optimisedoptimized

2. Duplicate paragraph in Gotchas

The sentence "These are GEO-specific behaviors that differ from standard doc authoring..." appears twice (lines 172 and 176). Remove one.

3. YAML frontmatter example has incorrect indentation

All top-level keys in the frontmatter template have two leading spaces ( id:, title:, etc.). Standard YAML frontmatter uses no leading indent on top-level keys. Anyone copying this example will get malformed frontmatter.

Minor

4. Description character count conflicts with style guide

The Gotchas section says "Keep description: under 160 characters." The style guide now specifies 140–160 as the target range — too short is also a problem. Update to: "Keep description: between 140–160 characters."

5. Comma splice (line 105)

"AI cannot follow cross-references between pages, a self-contained page is the only page that gets cited."

Should be two sentences.

6. Trailing space (line 78)

"matches the H1 (generated from frontmatter )" — space before the closing paren.

@kimsauce

Copy link
Copy Markdown
Collaborator

One more suggestion (non-blocking): consider adding a cross-references section at the bottom of the skill file pointing to the related tooling that now exists alongside this:

Without these pointers, someone reading the skill file has no way to discover the command workflow or the human-readable guidance in the style guide.

- Change British spelling to American (optimise → optimize)
- Remove duplicate paragraph in Gotchas section
- Fix YAML frontmatter indentation (remove leading spaces)
- Update description limit to 140–160 character range
- Fix comma splice in Principle 2
- Remove trailing space in frontmatter rules
- Add cross-references section linking to /geo-optimize, /seo-audit, and style guide

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
@JV0812 JV0812 requested review from kimsauce and removed request for jpipkin1 May 15, 2026 11:52

@kimsauce kimsauce left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Two structural issues to resolve before merge:

1. Skill directory name mismatch. The directory is geo-guide, which means Claude would invoke this as /geo-guide. But the active command listed in CLAUDE.md is /geo-optimize, which expects a skill at .claude/skills/geo-optimize/SKILL.md. If this is meant to back that command, rename the directory to geo-optimize.

2. CLAUDE.md not updated. All skills are explicitly listed in CLAUDE.md under ## Skills:

- .claude/skills/sumo-style/SKILL.md
- .claude/skills/docusaurus/SKILL.md
- .claude/skills/pr-template-guide/SKILL.md

This new skill needs to be added there so Claude loads it in future conversations.

Comment thread .claude/skills/geo-guide/SKILL.md Outdated
Comment thread .claude/skills/geo-guide/SKILL.md Outdated
Comment thread .claude/skills/geo-guide/SKILL.md Outdated
Comment thread .claude/skills/geo-guide/SKILL.md Outdated
Comment thread .claude/skills/geo-guide/SKILL.md Outdated
@JV0812

JV0812 commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator Author

@kimsauce I have addressed all your comments. As you said, we can update the file without the head frontmatter. Once we build the plumbing (engineering work) we can add those later.

@kimsauce kimsauce left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

The structure is solid and the 'Do not apply GEO for' section is especially important to get right — good call including it. A few things to tighten up below.

Comment thread CLAUDE.md Outdated
Comment thread .claude/skills/geo-guide/SKILL.md Outdated
Comment thread .claude/skills/geo-guide/SKILL.md Outdated
Comment thread .claude/skills/geo-guide/SKILL.md Outdated
JV0812 and others added 5 commits June 26, 2026 22:28
Co-authored-by: Kim <56411016+kimsauce@users.noreply.github.com>
Co-authored-by: Kim <56411016+kimsauce@users.noreply.github.com>
Co-authored-by: Kim <56411016+kimsauce@users.noreply.github.com>
Co-authored-by: Kim <56411016+kimsauce@users.noreply.github.com>
@JV0812 JV0812 enabled auto-merge June 26, 2026 17:01
@JV0812 JV0812 added this pull request to the merge queue Jun 26, 2026
Merged via the queue into main with commit e7a5f50 Jun 26, 2026
5 checks passed
@JV0812 JV0812 deleted the add-geo-guide branch June 26, 2026 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed Contributor approved, listed in .clabot file do-not-merge Pull requests that should not be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants