Repository navigation
NIST SP 800-53 and RMF: tags, citation rule, three skills, retags - #19
Merged
Merged
Conversation
- Framework tags nist-800-53 (NIST SP 800-53) and nist-rmf (NIST RMF) in the validator, the site labels and intros, the template and AGENTS.md. - NIST-tagged skills may name NIST control ids and families; the validator now matches 800-53 ids and enhancements, so requirement wording next to one is rejected like every other framework. Tests: bare id passes, id plus "requires" within 80 characters fails. - New house skills: system-categorization-first-pass, ssp-control-implementation-statements, poam-first-draft, each with an ISMS Copilot challenge record. - 14 existing skills retagged where their method serves NIST SP 800-53, RMF, SP 800-171 or CMMC practitioners; three gain one NIST sentence. - The site hides framework tags no skill carries; FAQ lists the new tags. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- ssp-control-implementation-statements: inheritance is evaluated under CMMC too (CMMC Assessment Process 2.17 to 2.21); an undocumented provider part is "undetermined", not hybrid; CMMC scoring per 32 CFR 170.24. - poam-first-draft: an item with no plan stays on the plan, escalated, until a recorded acceptance; acceptance never turns not met into met; operative 32 CFR 170.21 limits with the six ineligible ids by bare id. - incident-response-plan-drafting: DFARS 252.204-7012 trigger scope and preservation of monitoring and packet capture data. - Verified notes added to the Copilot challenge records; word headroom left in the three new skills. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- ssp-control-implementation-statements: both CMMC cloud paths (FedRAMP Marketplace or a FedRAMP Moderate equivalency body of evidence, CAP 2.20 and 2.21), limited to offerings handling CUI; implemented means evidence fit for the assessment objective, dated records only where a control recurs; CMMC findings met, not met, not applicable. - incident-response-plan-drafting: precise third DFARS 252.204-7012(c) trigger. - Verified notes updated in the challenge records. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… evidence, DFARS contract-identified trigger) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…nventions - system-categorization-first-pass, ssp-control-implementation-statements, poam-first-draft: data line in the inputs, Not checked and a blank Review record on the signed output (poam-review.md as the companion of poam.csv), NEVER fill in the Review record; old approval blocks removed. - incident-response-plan-drafting trimmed back under 1200 words after the rebase added the Not checked and Review record lines. - README, SKILL-LIST and CHANGELOG count 64 skills. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
tristanroth1
force-pushed
the
feat/nist-800-53-rmf
branch
from
October 27, 2026 10:22
81f23cf to
b3f8b97
Compare
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
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.
What and why
Adds NIST SP 800-53 and the NIST Risk Management Framework to the library, a promise made publicly after the launch post. It also fills the NIST SP 800-171 and CMMC shelves, which the tag list offered while zero skills carried them.
nist-800-53(NIST SP 800-53) andnist-rmf(NIST RMF) intools/validate.mjs, the site labels and framework intros, the template, AGENTS.md and the FAQ. Skill names still never contain NIST.nist-800-53,nist-rmf,nist-800-171ornist-csfmay name control ids and families (AC-2, the RA family). They never copy or paraphrase control wording. CMMC is unchanged. The rule is written in AGENTS.md (Ground rules), CONTRIBUTING.md (The bar), the template and the submit-skill system skill.CONTROL_IDgains an 800-53 id and enhancement pattern, so requirement wording within 80 characters of an id is rejected the same way as for every other framework. New validator tests: a bare id or enhancement passes;AC-2 requires ...fails; ids such asAC-2(3),SR-11andPM-9are matched.skills/tristan-roth/):system-categorization-first-pass(risk; nist-rmf, nist-800-53),ssp-control-implementation-statements(policies; nist-800-53, nist-rmf, nist-800-171, cmmc),poam-first-draft(audit; nist-800-53, nist-rmf, nist-800-171, cmmc). Each is checked against the hashed title denylist and has a Copilot challenge record inreviews/copilot-challenge/.loadContentdrops any framework tag no skill carries before the site uses it, so it never appears in list pages, filters, framework pages, the sitemap or OG images. The validator still accepts the tag. New site test insite/tests/content.test.mjs. After this PR every tag has at least 3 skills.Retagged skills (14)
risk-method-drafting+ nist-rmf: the method is the organizational risk strategy an RMF Prepare step builds on.risk-register-first-pass+ nist-rmf, nist-800-53, nist-800-171: scenario-based risk assessment works as written for the RA family and an 800-171 risk assessment.risk-acceptance-memo+ nist-rmf: one sentence added. The authorizing official owns acceptance of the system's remaining risk; link the memo to the plan of action item.incident-response-plan-drafting+ nist-800-53, nist-800-171, cmmc: the IR family is central. One clause added on DFARS 252.204-7012: which incidents trigger the report to DoD (72 hours), and preservation of images plus monitoring and packet capture data for at least 90 days.access-review-campaign-prep+ nist-800-53, nist-800-171, cmmc: account review is central to the AC family.logging-and-monitoring-evidence-pack+ nist-800-53, nist-800-171, cmmc: covers the AU family's evidence.change-management-evidence-sampling+ nist-800-53, nist-800-171, cmmc: covers the CM family's evidence.evidence-freshness-check+ nist-800-53, nist-800-171, cmmc: readiness of artefacts before an assessor samples them.internal-audit-checklist-from-scope+ nist-800-53, nist-800-171, cmmc: test per commitment with record, sample and pass criterion is a mock control assessment. The management-system tests are already conditional.vendor-tiering+ nist-800-53: supports supply chain risk work (SR family).first-gap-assessment+ nist-800-53, nist-800-171, cmmc: 800-171/CMMC readiness is a gap assessment. One sentence added: the SSP and the plan of action and milestones each get a row.multi-framework-overlap-map+ nist-800-53, nist-800-171: crosswalks to 800-53 are a core use.grc-setup-review+ nist-800-53, nist-rmf, nist-800-171, cmmc: it ranks the library for any setup, and the NIST skills now exist.backup-restore-test-evidence+ nist-800-53: restore testing belongs to the CP family.Considered and not tagged:
risk-treatment-plan-drafting(in NIST work the plan of action and milestones is the instrument, and the new skill covers it),incident-severity-triage(its clocks are EU and HIPAA only), and ISO-specific skills (SoA, stage 1, management review).Framework accuracy challenge
COPILOT_CHALLENGE: system-categorization-first-pass, conversation 114ef960-d4b0-49cc-a89a-2c542938e72e, Q1 PASS, Q2 FIX, Q3 FIX, all fixed (adjustment in both directions, security category wording, information owners, privacy threshold analysis, more triggers, CNSSI 1253 revision check)
COPILOT_CHALLENGE: ssp-control-implementation-statements, conversation 58849f7e-d587-4bb8-a1eb-5ad1de0555c5, Q1 PASS, Q2 FIX, Q3 FIX, fixed (Level 2 assesses against Rev. 2 per 32 CFR 170.14, CMMC met/not met per 170.24, enhancement rows, assessment objective check). Copilot's "no inherited for 800-171/CMMC" fix was reversed after Codex round 1: the CMMC Assessment Process v2.0 (2.17 to 2.21) evaluates inheritance claims, so the official text wins. Three additions declined, with reasons in the record
COPILOT_CHALLENGE: poam-first-draft, conversation d810629a-7f59-49ab-af5f-7603eb17c3cf, Q1 PASS, Q2 FIX, Q3 FIX, fixed (KEV timelines bind federal civilian agencies only, CMMC 180-day closeout and 170.21 limits, point value column, status list, SSP link). Copilot cited BOD 22-01, which cisa.gov shows as revoked and superseded by BOD 26-04, so the skill names "the directive in force". One claim (CMMC designations limited after a suspension) could not be verified and was not used
Checked against official text: 32 CFR 170.14(c)(3) and (d), 170.21 and 170.24 and DFARS 252.204-7012 (a), (c)(1) and (e) on eCFR; CMMC Assessment Process v2.0 sections 2.17 to 2.21 on cyberab.org; the BOD 22-01 revocation on cisa.gov. Each record has a Verified section.
Codex round 1 fixes
Codex round 2 fixes
Rebased onto #20 conventions
Rebased onto main (94f7f3e, after #18 and #20). The three new skills now carry the data line in their inputs, a run-specific
## Not checkedon the signed output, a blank## Review recordand the NEVER rule against filling it in. The signed outputs arecategorization-memo.md(names theinformation-types.csvversion),statements.md(names thestatement-index.csvversion) andpoam-review.md(the review companion ofpoam.csv, naming the CSV and the version or date approved). The old approval blocks were removed.incident-response-plan-draftingwas trimmed back under 1200 words. README, SKILL-LIST and CHANGELOG count 64 skills.Checks
node tools/validate.mjs: ok, 64 skillsnode --test tools/*.test.mjs: 182 pass, 0 failcd site && npm ci && npm run build: ok, 100 pages; SEO gate L0P: pass;npm test: 51 pass, 0 failCODEX_REVIEW: Codex 2026-10-27 round 1 FIX-FIRST (4 fixed), round 2 FIX-FIRST (4 fixed), round 3 narrow FIX-FIRST (3 wording fixes in 81f23cf); rebase confirmation: Codex unavailable (quota) -> Vibe 2026-10-27: SHIP
Owner: founder
Next: independent review by grcskills-feedback session, then merge and release
🤖 Generated with Claude Code