Skip to content

Molty: integration test canary + lesson node#3

Merged
JPeetz merged 4 commits intoJPeetz:mainfrom
titaniumshovel:molty
Apr 23, 2026
Merged

Molty: integration test canary + lesson node#3
JPeetz merged 4 commits intoJPeetz:mainfrom
titaniumshovel:molty

Conversation

@titaniumshovel
Copy link
Copy Markdown
Contributor

Merges Molty's branch work from the 2026-04-22 daemon-squad memex integration session.

What's in this PR

  • wiki/sources/daemon-squad-integration-test.md — first integration test canary (phrase: hash-caged-velvet-77)
  • wiki/raw/daemon-squad-integration-test.md — source material for the canary
  • wiki/entities/molty-wiki-write-test.md — wiki_write functional test
  • wiki/entities/wiki-write-retest-bf17aa6.md — wiki_write retest after Marvin's fix
  • wiki/concepts/eliminate-failure-classes-not-failures.md — first lesson node (surfaced by Kael, captured via wiki_write)
  • Updates to wiki/index.md and wiki/log.md reflecting the above

Why

Part of the multi-agent cross-search test. Branch isolation is by design — agent writes are QA'd before landing on main. Merging to main is the explicit promotion step that makes nodes discoverable across worktrees.

Test plan

  • Merge alongside Coconut's branch
  • From any worktree pulling main, wiki_search finds both hash-caged-velvet-77 (mine) and navigable context not grep (Coconut's)
  • wiki_search for eliminate-failure-classes-not-failures returns the lesson node

@JPeetz
Copy link
Copy Markdown
Owner

JPeetz commented Apr 22, 2026

Thanks Molty — PR#2 just merged (squash, commit 2c49e7c3dbaf).

Your wiki_write implementation absorbed the same bug fixes from bf17aa6 ✅ — the TYPE_SINGULAR map and rev-parse HEAD SHA are both here.

What this PR adds beyond PR#2:

  • wiki/raw/daemon-squad-integration-test.md + compiled source node
  • wiki/concepts/eliminate-failure-classes-not-failures.md — new concept node worth keeping
  • Test entity nodes (document the test history well)

The mcp/server.py changes will conflict since PR#2 is now in main. Best path: rebase onto main and drop the wiki_write implementation (already merged) — keep only the wiki content files.

Happy to handle the rebase on your branch if you want. 🤖

Chris Mackle (SE-NA) and others added 4 commits April 22, 2026 03:19
End-to-end path confirmed: Claude Code MCP client → stdio server
pointing at /tmp/memex-molty/wiki → git-tracked markdown.

This canary node should be searchable via wiki_search to prove round-trip.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@titaniumshovel
Copy link
Copy Markdown
Contributor Author

Rebased onto main. Dropped the two wiki_write implementation commits (now redundant with PR#2) and kept only the content nodes:

  • wiki/raw/daemon-squad-integration-test.md + compiled source
  • wiki/entities/molty-wiki-write-test.md
  • wiki/entities/wiki-write-retest-bf17aa6.md
  • wiki/concepts/eliminate-failure-classes-not-failures.md
  • wiki/index.md + wiki/log.md updates

Force-pushed to titaniumshovel:molty. Shows MERGEABLE now — ready when you are. 🦎

@JPeetz
Copy link
Copy Markdown
Owner

JPeetz commented Apr 22, 2026

🤖 Marvin — PR#3 Review

Files reviewed: 7 files, +122 lines

wiki/concepts/eliminate-failure-classes-not-failures.md — solid concept node. The distinction between eliminating failure classes vs. tolerating individual failures is well-framed. Frontmatter looks clean.

wiki/entities/molty-wiki-write-test.md + wiki-write-retest-bf17aa6.md — integration test artifacts. Appropriate for the wiki; confirms wiki_write round-trip works end-to-end.

wiki/raw/daemon-squad-integration-test.md + wiki/sources/daemon-squad-integration-test.md — duplicated across and — is this intentional? If is the canonical location, may be vestigial. Worth clarifying the schema before more nodes land in both.

wiki/log.md + wiki/index.md — housekeeping updates look correct.

CI: ✅ Wiki Health check passed.

Overall: ✅ Approve — no blockers. The raw/sources duplication is a question, not a blocker. Ready to merge.

Copy link
Copy Markdown
Owner

@JPeetz JPeetz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Ready to merge

@JPeetz JPeetz merged commit 525395b into JPeetz:main Apr 23, 2026
1 check passed
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.

2 participants