Decisions

  • Pending: Review frequency (weekly / triggered / both)?
  • Pending: Scope — config only or include vault health?
  • Pending: Health check trigger — /health skill, daily briefing, or both?
  • Decided: Use YAML frontmatter sync-frequency field per file to control how often docs are checked for freshness

User Tasks


Summary

Opus periodically reviews and improves its own configuration — agents, skills, hooks, rules, CLAUDE.md, and all Claude Code features.

Problem / Motivation

Claude Code evolves fast. New features (agents, skills, hooks, MCP servers, teams, cowork, planning mode) get added regularly. Without periodic review, the Opus setup becomes outdated and doesn’t leverage what’s available.

Proposed Solution

A scheduled self-review routine that audits the entire Opus configuration against current Claude Code capabilities, and proposes improvements.

What Gets Reviewed:

ComponentLocationReview Question
CLAUDE.md/CLAUDE.mdStill accurate? Missing new conventions?
Skills.claude/skills/Any new skills needed? Existing ones working well?
Hooks.claude/hooks/ + settings.jsonAny new hook events useful? Current hooks effective?
Agents.claude/agents/Agent definitions still optimal? New agent types available?
Rules.claude/rules/Rules still relevant? Need new path-specific rules?
MCP Servers.mcp.jsonNew integrations available? Current ones working?
Settings.claude/settings.jsonPermissions correct? New settings available?
Architecturevault/00_system/System docs still match reality?
Docs (all)vault/00_system/docs/, dashboards, etc.Content still accurate? Needs update? Check sync-frequency frontmatter for schedule.

Health Checks (Smoke Tests):

Periodic tests that verify working capabilities haven’t broken. Each capability in capabilities-dashboard gets a health check that Claude can run.

CheckWhat it testsHow
/idea skillCapture pipeline worksRun skill, verify file created, clean up
/status skillStatus overview worksRun skill, verify output
Vault structureAll expected folders existGlob for required paths
Git integrationRepo healthy, remote reachablegit status, git remote -v
Hook scriptsHooks execute without errorTrigger hook, check exit code
Template integrityAll templates exist and validCheck _templates/ files have frontmatter
Atlas accessCan read Atlas vault (once FR-062 done)Read a known Atlas path
English-only checkAll vault and code files are written in EnglishScan files for non-English content, report violations
Token efficiencyCLAUDE.md, rules, skills, agents not bloatedCheck file sizes, flag verbose or redundant content, propose trimming
Code qualityCodebase stays clean (complements FR-067)Flag dead code, unused imports, stale configs
Broken wikilinksNo [[links]] pointing to non-existent filesScan vault for wikilinks, verify targets exist
Stale FRsNo feature requests untouched for 30+ daysCheck updated frontmatter, flag stale ones
Uncommitted changesNo large git diffs piling up without commitgit status + git diff --stat, warn if excessive
Secrets scanNo API keys, passwords, or .env content in repoScan for common secret patterns in tracked files
MCP server healthConfigured MCP servers still respondingCheck .mcp.json entries, verify connectivity
Orphan filesNo files unlinked from any dashboard, FR, or docCross-reference all files against links/references
Decision log freshnessNo significant decisions missing from decision-log.mdCompare recent git history and FR decision checkboxes against log entries
Protected file integrityNo unauthorized changes to edit-policy: strict filesHash or git-diff protected files against known-good state (complements FR-005)

Results feed back into capabilities-dashboard as [passing] / [failing] tags.


Open Questions

No open questions.


Phase Overview

PhaseDescriptionStatus
Phase 1/self-review skill + checklist
Phase 2FR test runner — scan and run all FR test sections
Phase 3Automated audit
Phase 4Self-implementation

Phase 1: /self-review Skill + Checklist —

Goal: Create a manual self-review skill that audits the system and produces findings.

File / FeatureDetailsOwnerStatus
/self-review skill.claude/skills/self-review/SKILL.mdmv
/health skillRun all health checks, report resultsopus
Health check definitionsDefine checks per capability (see table above)opus
Recurring task detectionCheck for patterns that should become skillsopus
Tool usage auditReview current tool/hook/agent usagemv
Knowledge gap checkFlag stale proposals and outdated docsopus
Config auditCLAUDE.md, skills, hooks, agents, rules, MCP, settingsopus
Review checklist templateStandardized checklist for reviewsmv
Run scheduleManually once per weekmv

Phase 2: FR Test Runner —

Goal: Systematically scan all FRs for ## Test sections and run them periodically.

File / FeatureDetailsOwnerStatus
FR test scannerScan all FRs in 10_features/ for ## Test sectionsmv
Classify testsDetermine which tests are automatable (script) vs manual (user)mv
Run automated testsExecute automatable tests and write results back to the FRmv
Report manual testsSurface manual tests that are due to the usermv
ScheduleRun as part of health check or briefingopus

Phase 3: Automated Audit —

Goal: Script that automatically checks for new Claude Code features and compares against current config.

File / FeatureDetailsOwnerStatus
Doc checker scriptChecks Claude Code docs for new featuresopus
Docs freshness cronScheduled task that reviews all system docs (architecture, dashboards, docs/) against actual state. Proposes updates to user — never auto-applies.mv
sync-frequency frontmatterEach doc gets a `sync-frequency: weeklyopusmonthly
Config comparisonCompare current config against available optionsopus
Proposal generationGenerate improvement proposalsopus
Human approvalPresent proposals to user for approval — all doc/config changes require explicit user sign-offmv

Phase 4: Self-Implementation —

Goal: After human approval, implement changes automatically with rollback capability.

File / FeatureDetailsOwnerStatus
Auto-implementApply approved proposalsmv
Test changesVerify the change worksmv
RollbackRevert if tests failmv

Prerequisites / Gap Analysis

Requirements

RequirementDescription
REQ-1FR-038 (Claude Manual) — knowledge base for comparison

Current State

ComponentStatusDetails
Skills infrastructuredone.claude/skills/ exists
Claude Code manualFR-038 not yet built

Gap (What’s missing?)

GapEffortBlocker?
Self-review skillMedNo
Claude Code manual for referenceMedNo (can work without, better with)

Test

Manual tests

TestExpectedActualLast
Weekly review produces improvement reportReport with findingspending-

AI-verified tests

ScenarioExpected behaviorVerification method

E2E tests

ScenarioAssertion

Integration tests

ComponentCoverage

Unit tests

ComponentTestsCoverage

Note: This FR merges the former FR-048 (System Self-Review) and FR-048 (Periodic System Review).

History

DateEventDetails
2026-02-26CreatedFrom brain dump
2026-02-27RenumberedFrom FR-013 to FR-012
2026-02-28ReformattedAligned to feature-request template

References

  • FR-038 (Claude Manual) — the knowledge base, this is the action
  • FR-067 (Auto Cleaning) — vault/code hygiene, this is config evolution
  • FR-012 (Periodic System Review) — complementary: docs consistency vs config evolution
  • FR-005 (Protected Files) — edit-policy frontmatter complements sync-frequency; both use YAML metadata per file
  • ClaudeClaw Rebuild Prompt — status/health check script pattern (npm run status)
  • FR-029 (Task Scheduler) — provides scheduled execution runtime