chore: persist agent memory into repo docs and skills - #293
Open
vpetersson wants to merge 2 commits into
Open
Conversation
Extract accumulated Claude Code session memory into committed files before the dev VM is rotated: - CLAUDE.md: git commit-identity convention (env vars, never git config), wizard TUI fit-to-viewport/responsive rules, user-facing help-text guidance (compliance framing + ../sbomify.com content as source of truth), Dependabot exclude-paths notes, and a pointer to docs/plans/ for in-flight plans - .claude/skills/copilot-review/SKILL.md: Copilot review-round workflow (batch commit, REST re-request review, commit-and-ping replies, resolve threads only on explicit request) - docs/plans/yocto-textual-migration.md: confirmed decisions and full implementation plan for migrating the yocto command onto the wizard's Textual framework (not yet implemented) - .gitignore: narrow the .claude ignore so committed skills are tracked while local state (settings, worktrees) stays ignored Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Persists previously accumulated agent/session guidance into the repository as durable documentation and reusable “skills”, so future work in this codebase follows the same operational conventions (commit identity, Textual wizard constraints, Dependabot behavior, and Copilot review workflow).
Changes:
- Adds a Yocto→Textual migration plan document under
docs/plans/. - Extends
CLAUDE.mdwith operational/dev-process guidance (git identity, wizard TUI rules, help-text framing, Dependabot notes). - Adjusts
.gitignoreto keep.claude/skills/tracked while ignoring other local.claudestate, and adds a Copilot-review workflow skill doc.
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
docs/plans/yocto-textual-migration.md |
Adds an implementation plan for migrating the yocto command onto the Textual-based wizard framework. |
CLAUDE.md |
Documents repository-specific development rules and operational conventions (git identity, TUI design, help text, Dependabot). |
.gitignore |
Narrows .claude ignore rules to allow committing skills while ignoring local agent state. |
.claude/skills/copilot-review/SKILL.md |
Adds a procedural skill for handling GitHub Copilot review rounds via REST review re-request + per-thread replies. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
The plan will be tracked as a GitHub issue instead of a committed doc. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Comment on lines
+86
to
+88
| ## Git Commit Identity | ||
|
|
||
| Worktree environments may have no inherited `user.name`/`user.email`. **Never run `git config` to set identity.** Pass identity via environment variables on the commit command instead: |
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.
Before rotating the dev VM, this extracts the accumulated Claude Code session memory into files that live in the repo (all personal details — names, emails, session IDs — scrubbed):
CLAUDE.md additions
GIT_AUTHOR_*/GIT_COMMITTER_*env vars, nevergit config; match the feature-branch author identity fromgit log, not the squash-merge noreply identity.-tiny/-compact/-no-art/-no-previewresponsive class system inscreens/_base.py+styles.tcss,wizard-help/wizard-mutedclasses,PagedFormScreenfor long forms, 80×24 minimum.../sbomify.comHugo content as source of truth when available.exclude-pathsexists (Aug 2025) for fixture noise; doesn't affect security PRs; uv graph builder ignores it (uv dependency graph doesn't respectexclude-pathsoption dependabot/dependabot-core#15102).New skill
.claude/skills/copilot-review/SKILL.md— the Copilot review-round workflow: batch fixes into one commit, re-request the review via the RESTrequested_reviewersendpoint with thecopilot-pull-request-reviewer[bot]slug (@copilotcomments andgh pr edit --add-reviewerdo NOT work), commit-and-ping per-thread replies, and leave thread resolution to the reviewer unless explicitly told otherwise..gitignoreis narrowed from.claudeto.claude/*+!.claude/skills/so committed skills are tracked while local state stays ignored.The in-flight yocto → Textual migration plan is tracked as #294 instead of a committed doc.
🤖 Generated with Claude Code