Skip to content

chore(skills): cut the rfc skill to a 1-2 pager - #511

Merged
filvecchiato merged 2 commits into
mainfrom
docs/agents-md
Aug 26, 2026
Merged

chore(skills): cut the rfc skill to a 1-2 pager#511
filvecchiato merged 2 commits into
mainfrom
docs/agents-md

Conversation

@filvecchiato

Copy link
Copy Markdown
Collaborator

The rfc skill targets a 1-2 page RFC that settles the approach and leaves the
details to the implementer.

196 → 66 lines. Gone: the mandatory rounds of clarifying questions, the ten-area
clarity checklist, the hand-it-back-for-review phase, and the Polkadot Fellows
template's Stakeholders, Testing/Security/Privacy,
Performance/Ergonomics/Compatibility, Prior Art and Future Directions sections.
Drawbacks and Alternatives merge into one Trade-offs section.

Template now matches number-rfc.yml

The old template did not, in three ways worth knowing about independently of the
brevity change:

  • The H1 is # RFC — Title with an em dash, which is the exact form the
    numbering step rewrites (s/^# RFC — /# RFC NNNN — /). # RFC: Feature Name Here matched neither pattern, so the assigned number was never injected into
    the title.
  • Frontmatter carries title / owner / status where extract_meta reads
    them, and status: draft is set explicitly: a missing status is defaulted to
    accepted, so a draft would have been indexed as accepted.
  • The four-row metadata table is dropped, since extract_meta prefers
    frontmatter. That also removes a hardcoded author name that would have
    mis-attributed every RFC generated from the template.

The skill now says not to number the file and not to touch _index.md — CI does
both on merge to main.

Not addressed here

check-rfc.yml fails any PR touching docs/rfcs/** that does not also change
rust/crates/truapi/. A host-side proposal cannot satisfy that, since the crate
invariants keep host-side runtime types in truapi-platform/truapi-server, so
such a proposal has to be filed under docs/features/ instead. The skill
documents the constraint; removing it is a separate call about the process rather
than the skill.

Details are the implementer's. Template now matches number-rfc.yml: em-dash
H1 for number injection, frontmatter status set so CI does not index a draft
as accepted.
@filvecchiato
filvecchiato requested a review from a team August 26, 2026 09:16
@filvecchiato
filvecchiato enabled auto-merge August 26, 2026 09:33
@filvecchiato
filvecchiato added this pull request to the merge queue Aug 26, 2026
Merged via the queue into main with commit 5894776 Aug 26, 2026
17 checks passed
@filvecchiato
filvecchiato deleted the docs/agents-md branch August 26, 2026 09:48
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.

2 participants