Skip to content

chore: bring this repo's own spanweave wiring current (SessionEnd + full read-pointer) - #113

Merged
ooiyeefei merged 1 commit into
mainfrom
chore/dogfood-own-wiring-current
May 29, 2026
Merged

ooiyeefei merged 1 commit into
mainfrom
chore/dogfood-own-wiring-current

Conversation

@ooiyeefei

@ooiyeefei ooiyeefei commented May 29, 2026 •

Copy link
Copy Markdown
Owner

Dogfooding caught config drift in spanweave's own repo:

Both were produced by running spanweave init --tool claude-code in this repo (the migration paths from #108 and #96 doing their job), then tidying the leftover empty Stop: [].

No code change — config only. The tool's own repo should reflect its current behavior, especially before a demo that may show this repo.

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Documentation

    • Added session startup instructions to load previous context and decision history.
  • Chores

    • Updated session hook event configuration naming.

Review Change Stack

Dogfooding caught drift: this repo's own .claude/settings.json still used the
pre-#108 `Stop` hook event, and CLAUDE.md still had the pre-#96
decisions-only read-pointer. A `spanweave init` run brought both current:
- hook migrated Stop -> SessionEnd (fires once per session, not per turn)
- read-pointer now covers all record-type dirs (decisions/findings/
  reflections/rejected_alternatives + their shared/ copies)

The tool's own repo should reflect its latest behavior — especially before
recording a demo that may show this very repo.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented May 29, 2026 •

Copy link
Copy Markdown
ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Free

Run ID: 21a4a9da-01e6-499b-a646-6bbe2177bc0b

📥 Commits

Reviewing files that changed from the base of the PR and between 1ba4ba0 and caf90b2.

📒 Files selected for processing (2)
  • .claude/settings.json
  • CLAUDE.md

📝 Walkthrough

Walkthrough

The pull request updates session lifecycle management by renaming the hook event key from Stop to SessionEnd in the Claude configuration, and adds documentation instructing the assistant to load prior Spanweave context (decisions, findings, reflections) at session start.

Changes

Session Lifecycle Management

Layer / File(s) Summary
Session lifecycle configuration and documentation
.claude/settings.json, CLAUDE.md
Hook event renamed from Stop to SessionEnd in configuration. Documentation section added specifying which .spanweave/memory/ subdirectories (decisions, shared decisions, findings, reflections, rejected alternatives) to load on session start before substantive work.

🎯 1 (Trivial) | ⏱️ ~3 minutes

A rabbit hops through the settings with glee,
SessionEnd marks where memories shall be,
Load the Spanweave, decisions so bright,
Before the work starts, context takes flight! 🐰✨


Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

@ooiyeefei
ooiyeefei merged commit 0b76542 into main May 29, 2026
7 checks passed
@ooiyeefei
ooiyeefei deleted the chore/dogfood-own-wiring-current branch May 29, 2026 15:10
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.

1 participant