Conversation
…ase notes - Consolidate 5 release-note-* commands into a single release-note.md - Remove "What this command does", "When to use", "Post-completion message", "Tips", "After the X", "Example usage", and generic error-handling sections from audit-doc, doc, doc-from-jira, geo-optimize, remove-doc, rewrite-intro, seo-audit, simplify, and tone-check commands - Update CLAUDE.md slash command table to reference /release-note Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Applies audit-doc checks to all changed .md files in a PR and produces a consolidated review comment. Read-only — surfaces issues without fixing. Also fixes sentence case rule in audit-doc.md (was incorrectly Title Case). Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Rules are covered by the sumo-style skill. Standalone commands were unlikely to be invoked and added clutter to the command list. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
kimsauce
changed the title
kimsauce
requested review from
JV0812,
amee-sumo,
mafsumo and
vfalconisumo
as code owners
* DOCS-1634 - Fix MCP server and OAuth docs (#6710) * DOCS-1634 - Fix MCP server and OAuth docs - Remove VS Code and ChatGPT sections from MCP server doc; scope prereqs to any OAuth 2.0 Authorization Code-compatible client - Fix claude mcp add command syntax for --client-secret flag - Expand OAuth client setup steps inline for Claude Code CLI - Soften prerequisite language in both docs - Rewrite Auth Code flow Step 2 to be actionable - Replace hardcoded deployment URLs with <token-endpoint-URL> and <api-endpoint> placeholders in Client Credentials Step 3 - Add deployment token endpoint table to FAQ - Fix /oauth/token typo to /oauth2/token throughout Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> * DOCS-1634: Add Known limitations section to MCP server doc Documents that Cursor (cursor:// redirect URLs unsupported) and recent VS Code releases (auth code flow broken with explicit client credentials) are not currently supported. Includes a note to contact support. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> * Bump webpack-dev-server from 5.2.2 to 5.2.4 (#6718) Bumps [webpack-dev-server](https://github.com/webpack/webpack-dev-server) from 5.2.2 to 5.2.4. - [Release notes](https://github.com/webpack/webpack-dev-server/releases) - [Changelog](https://github.com/webpack/webpack-dev-server/blob/main/CHANGELOG.md) - [Commits](webpack/webpack-dev-server@v5.2.2...v5.2.4) --- updated-dependencies: - dependency-name: webpack-dev-server dependency-version: 5.2.4 dependency-type: indirect ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * DOCS-1526 - Add Bulk Changes guardrails to CLAUDE.md Distill the DOCS-63 post-mortem lessons into a short CLAUDE.md section instead of a full bulk-doc-change skill. Newer models handle most of the workflow naturally; only the non-obvious institutional rules (no revert from memory, atomic commits per directory, no committed helper scripts) need to persist as durable guidance. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com> --------- Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
kimsauce
changed the title
- audit-doc, doc, seo-audit: replace generic "What this command does" steps with imperative descriptions and a Gotchas section - audit-doc: fix heading case rule (H1 Title Case, H2–H4 sentence case); restore redundant when-to-use bullet - seo-audit: restore dropped checks (anchor text, bare URLs, missing alt, Critical table rows) - CLAUDE.md: add plan mode requirement for bulk changes; clarify bulk change examples with e.g.; remove opaque DOCS-63 reference Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Added guidelines for branch naming and commit approval process.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Purpose of this pull request
Cleans up the Claude Code setup in this repo — removes redundant commands, deduplicates overlapping checks between
/audit-docand/seo-audit, fixes a heading case bug, and adds durable institutional guardrails for bulk doc changes.Deleted 3 redundant style commands
Deleted
/tone-check,/simplify, and/rewrite-intro— the rules they contained are already covered by thesumo-styleskill and applied automatically on every doc edit. Standalone slash commands were unlikely to be invoked directly.Removed corresponding rows from the
CLAUDE.mdcommand table.Bug fix in
audit-doc.mdCorrected heading case rule from "Title Case" to "sentence case" to match actual repo convention (titles use Title Case; H2+ headings use sentence case).
Dead weight stripped from
audit-doc.mdanddoc.mdRemoved low-value sections from both files: scripted example dialogues, post-completion messages, tips and best practices, error handling for edge cases, and references — none of these affect runtime behavior.
## What this command doesand## When to use this commandare kept — these provide context for when to proactively suggest each command.Deduplicated
seo-audit.mdRemoved checks already covered by
/audit-doc:titlepresentdescriptionpresent/seo-auditnow focuses exclusively on SEO/AEO/GEO signals that/audit-docdoes not cover.Also removed two stale references to the deleted
/tone-checkcommand.New Bulk Changes section —
CLAUDE.mdAdds a short Bulk Changes section capturing institutional guardrails from the DOCS-63 post-mortem: don't revert from memory, atomic commits per directory, no committed helper scripts, dry-run before writing.
Chosen over a
/bulk-doc-changeskill (originally proposed in DOCS-1526) because newer Claude models handle the workflow naturally — the unique value is the guardrails, not a five-step wrapper.Select the type of change
Ticket (if applicable)
https://sumologic.atlassian.net/browse/DOCS-1640
https://sumologic.atlassian.net/browse/DOCS-1526