Repository navigation
🐛 [fix] state: keep board state inside a submodule's own repo - #88
Merged
Merged
Conversation
- test_github_reads: a halt in projects/a leaves projects/b and the parent running - test-card: a submodule's card cache lives in its own git dir - both fail on main: state lands in the shared .git/modules/<parent>/.claude/ Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
- halt_path() and card cache_dir use <git-common-dir>/super-board/ unless the common dir is <root>/.git - normal repos and linked worktrees keep <main checkout>/.claude/super-board/ - run-workflow.md names the submodule location; ported from EricOS 3d252cc91 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
EricTechPro
added a commit
that referenced
this pull request
Oct 29, 2026
- notes name the VERSION stamp on the public site - the three EricOS fork ports each get a line with their PR Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Merged
8 of 9 tasks
EricTechPro
added a commit
that referenced
this pull request
Oct 29, 2026
…l map refresh (3.3.0) (#86) * ✨ [feat] diagrams: security-audit map page and skill-map view - New /security-audit/ page: Cloudflare's security-audit skill (commit c1c8a8c) on the shared sdlc layout, six phases, 63 parts with when/what/how/behind-the-scenes popovers and line-linked sources. - layout_sdlc: optional meta.labels, per-stage name and icon, and `parallel` categories (no arrows, no START); SDLC maps rebuilt with identical diagrams. - Registry, nav, Explore card, 960x600 thumb, Cloudflare logo, pages.yml and README link. - skill-map.json: external security-audit node in its own root child view (six main steps), re-rendered with 0 crossings; visual tests pass. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * 🔒 [security] review: run pr-security-review on every PR, install security-audit (3.2.0) - new bundled skill pr-security-review: Anthropic's local /security-review, prompt verbatim, diff target = PR base (MIT notice + source commit 0c6a49f) - super-review runs it on every PR; findings are Bug · blocker routed to Super Build - get.sh and onboard repair fetch cloudflare/security-audit-skill via npx skills; skills-lock.json detection, --no-helper-skills respected - skills-ready: pr-security-review required in Review, security-audit optional - super-qa drops its /security-review call; README, main page, skill map, simulator, manifests, onboard.md updated - tests: get.sh security-audit cases, onboard repair order, readiness entries Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * 🔒 [security] qa: run pr-security-review in QA, not Review - super-qa runs pr-security-review on every PR as its one security check (folds the old /security-review call) - each kept finding fails QA: ❌ line, [builder] thread, QA → Ready; skill missing → Block - super-review drops its security step; the rest of the lane is unchanged - skills-ready: pr-security-review required in qa, security-audit optional in qa; docs, families, manifests, README synced Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * ✨ [feat] collect: hand verified findings to /to-spec and /to-tickets - new collect_handoff.py: route (to-spec/to-tickets installed?), doc (one grouped findings doc), fork (1–5 → /implement-spec, 6+ → Backlog) - findings doc at docs/super-board/findings/collect-<YYYY-MM-DD>.md, never overwritten; dry run prints it and writes nothing - missing /to-spec or /to-tickets, or --direct → today's Backlog filing, with the reason printed; scheduled intake uses --direct - tests/test_collect_handoff.py; SKILL.md, README, docs flow and AGENTS.md block updated Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * 📝 [docs] routing: write down which path to take, in plain words - docs/super-board/routing.md: clear idea, foggy idea, bugs → /to-spec → /to-tickets - build fork: 1–5 tickets → /implement-spec, 6+ → super-board; super-deploy planned - linked from README and the super-board SKILL.md Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * ✨ [feat] diagrams: two doors and a route fork on the skill map root - root: Ideas (/grill-me, /wayfinder) and Bugs (/super-collect) meet at /to-spec → /to-tickets; 1–5 tickets → /implement-spec, 6+ → /super-board; planned super-deploy; /security-audit joins the side tools - pr-security-review moves from super-review to super-qa; super-collect hands a findings doc to /to-spec - visual template: When · What · How · Example · Behind the scenes tabs, ==highlight==, examples gallery, planned nodes, loop edges, wrapping bullets on phones - tests: test_visual_* pass (families and main-steps updated for the new root) Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * 🔧 [chore] release: 3.3.0 notes and version - VERSION, skills/super-board/VERSION, plugin.json and README badge → 3.3.0 - RELEASE-NOTES: security check in QA, super-collect planning hand-off, routing doc - bash tests/run-safety.sh: 63 suites pass Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * ✨ [feat] diagrams: four /visual examples in the /visual dialog - examples: PR description (recap of this branch), super-deploy plan, architecture map, command and script interface - each has a full-size PNG and a thumb under 100 KB; Pages serves them at /skill-map/examples/ - landing thumb redrawn from the new root view; sync_explore --check passes Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * 📝 [docs] diagrams: super-collect hand-off view matches the code - collect-submit view: findings doc, /to-spec, /to-tickets, then 1–5 to /implement-spec on yes, 6+ to Backlog - --direct, scheduled runs, GitHub issues and a missing /to-spec or /to-tickets go straight to Backlog - collect-file node and its edges reworded to match SKILL.md and collect_handoff.py - re-rendered skill-map.html and index.html; test_visual_*, visual check, sync_explore and run-safety pass Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * 🧪 [test] pages: live site shows repo VERSION, not a typed release - fails today: index, onboarding and writing-standard hardcode v3.0.2 - pins the dev placeholder, the build stamp and the VERSION path trigger Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * 👷 [ci] pages: stamp VERSION into the site at build - data-sb-version placeholders read "dev" from disk, stamped at deploy - pages.yml runs stamp_version.py and redeploys when VERSION changes - release pill links to releases/latest Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> * 📝 [docs] release: 3.3.0 notes list the pages stamp and #88 #89 #90 - notes name the VERSION stamp on the public site - the three EricOS fork ports each get a line with their PR Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
EricTechPro
added a commit
that referenced
this pull request
Oct 29, 2026
- merge: non-Claude hosts merge by hand unless auto is set (#47, 7859103) - lane: one generic launcher, super-board-lane.sh --backend (cc38679) - setup: back-to-back fixes no longer crash on the backup folder (76e8ac5) - #88, #89, #90 were already listed Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Note
⏳ In review · round 1 · ✅ 3/3 AC · head
aa719e9Problem
projects/<name>).git/modules/<path>, so "parent of the common dir" put every sibling'sgithub-halt.jsonand cardcache/in one shared.git/modules/<parent>/.claude/. One project's halt paused the others.Solution
halt_path()(super-board-github-read.py) andcache_dir(super-board-card.sh) use<main checkout>/.claude/super-board/only when the common dir is<root>/.git; otherwise<git-common-dir>/super-board/SB_GITHUB_HALT_FILEandSB_CARD_CACHE_DIRstill overridesuper-board-collect-state.pyfollowshalt_path(), so collector state moves with itrun-workflow.mdnames the submodule location3d252cc91); no VERSION or notes change here, they go in the 3.3.0 notes on ✨ [feat] routes: two doors into the board, security check in QA, skill map refresh (3.3.0) #86Acceptance criteria
↳
test_github_reads.py::test_submodules_keep_their_own_halt· failed ona2809b6, passes now↳
tests/test-card.shcheck 10 · failed ona2809b6("submodule cache not in its own git dir"), 10/10 now↳
bash tests/run-safety.sh· 62 suites pass on macOSIteration history
aa719e9Risk
🟢 Low · only submodule installs see a new state path.
🤖 Generated with Claude Code