Skip to content

chore: agent-agnostic standards, workflows, and skill optimization#2366

Open
TC-MO wants to merge 5 commits intochore/strengthen-vale-toolingfrom
chore/agent-agnostic-standards-and-workflows
Open

chore: agent-agnostic standards, workflows, and skill optimization#2366
TC-MO wants to merge 5 commits intochore/strengthen-vale-toolingfrom
chore/agent-agnostic-standards-and-workflows

Conversation

@TC-MO
Copy link
Copy Markdown
Contributor

@TC-MO TC-MO commented Mar 24, 2026

Draft - writeup in notion

Summary

  • Move .claude/rules/standards/ at repo root (agent-agnostic)
  • Extract workflows and references from .claude/skills/workflows/ at repo root
  • Rewrite all 4 skills as summary + pointer adapters with AgentSkills spec compliance
  • Peg review-docs to Sonnet with hybrid subagent approach (eval-driven decision)
  • Delete Cursor symlinks, rewrite .mdc files as thin pointers
  • Add lint ignores for agent-facing infrastructure files

Depends on #2365

🤖 Generated with Claude Code

TC-MO and others added 3 commits March 24, 2026 14:01
Move .claude/rules/ to standards/ at repo root so all agents have
parity. Replace verbose style sections in AGENTS.md with condensed
summary + pointers. Trim standards where Vale now enforces the check.
Add information ordering principle. Update symlinks, supporting files,
and lint ignores.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Move workflow processes and references from .claude/skills/ to
workflows/ at repo root for agent parity. Rewrite all 4 SKILL.md
files as summary + pointer adapters with AgentSkills spec compliance.
Delete Cursor symlinks, rewrite .mdc files as thin pointers.
Add check-frontmatter.sh script and workflows section to AGENTS.md.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Eval showed Sonnet hybrid (summary + pointer adapter with delegated
standards subagents) is the best tradeoff: ~80% cheaper than old Opus
approach, 4% faster, and catches more issues. Deterministic tools run
in main process (no sandbox issues), subagents handle standards review.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@TC-MO TC-MO self-assigned this Mar 24, 2026
@TC-MO TC-MO added the t-docs Issues owned by technical writing team. label Mar 24, 2026
@apify-service-account
Copy link
Copy Markdown

Preview for this PR was built for commit 3e122baa and is ready at https://pr-2366.preview.docs.apify.com!

description: Create structured tutorials for Apify Academy or Platform documentation. Use when user says "create a tutorial", "write a tutorial", "build a step-by-step guide", "convert this guide into a tutorial", or "tutorial for [topic]". Handles tutorial structure, learning objectives, prerequisites, step-by-step instructions, code examples, and troubleshooting sections.
argument-hint: topic
allowed-tools: Read, Write, Edit, Bash, Glob, Grep
allowed-tools: Read Write Edit Bash Glob Grep
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why not use comma separated values?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

to be spec compliant with agent skill spec. Claude accepts both so to make them more agent-agnostic we switch to spaces

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

While we're at it, why not axe this file comletely? It's only mentioned in the claude.json file, but that is a halucinated file that is not loaded as standard.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

yup good catch, both claude.json and instructions.md to be axed

argument-hint: endpoint-path
allowed-tools: Read, Write, Edit, Bash, Glob, Grep
allowed-tools: Read Write Edit Bash Glob Grep
metadata:
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I didn't try this yet, but is this nesting supposed to be there?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

nice, I think this might have to be reverted. Agent Skills spec nests argument-hint within metadata, but Claude Code ignores metadata completely

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

honestly Anthropic not being compliant with an open standard trips me for I think 3rd time, where I just assume that they are, but in reality they aren't :D

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Even funnier is they actually are responsible for the agent-spec yet don't follow it

argument-hint: file-path
allowed-tools: Read, Bash, Glob, Grep, Agent
allowed-tools: Read Bash Glob Grep Agent
model: sonnet
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why tho? I wouldn't force a model here. Sonnet is often not capable enough to catch many issues.

Copy link
Copy Markdown
Contributor Author

@TC-MO TC-MO Mar 25, 2026

Choose a reason for hiding this comment

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

Did you actually went through the writeup in notion? I tested this approach with multiple models & sub-agents and no sub-agents. Sonnet was the most competent of the bunch in actually finding the issues. Opus while good for generation missed a lot and is prone to overthinking.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I just quickly want through the PR, will check notion

Move argument-hint out of metadata: block back to top-level - Claude
Code doesn't recognize metadata: nesting. Delete claude.json (not a
real Claude Code config) and instructions.md (only referenced by
claude.json). PR review feedback from marcel-rbro.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@apify-service-account
Copy link
Copy Markdown

Preview for this PR was built for commit dd777144 and is ready at https://pr-2366.preview.docs.apify.com!

…rkflows

Skills now live in `.agents/skills/` (the AgentSkills open standard path),
discoverable by Codex, Gemini CLI, OpenCode, Cursor, and other compatible
agents. Claude Code discovers them via symlinks in `.claude/skills/`.

Workflow content (process.md, references/, scripts/) absorbed into each
skill's own `references/` and `scripts/` directories per the spec structure.
The `workflows/` directory is deleted. Standards remain at repo root (shared).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@apify-service-account
Copy link
Copy Markdown

Preview for this PR was built for commit e99697cc and is ready at https://pr-2366.preview.docs.apify.com!

@TC-MO TC-MO marked this pull request as ready for review March 31, 2026 13:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

t-docs Issues owned by technical writing team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants