Skip to content

Pull in upstream #186, #168, #172, #179 - #1

Merged
AndrewSpell merged 5 commits into
mainfrom
upstream-picks-186-168-172-179
Oct 10, 2026
Merged

AndrewSpell merged 5 commits into
mainfrom
upstream-picks-186-168-172-179

Conversation

@AndrewSpell

Copy link
Copy Markdown
Owner

Four open upstream PRs from obra/episodic-memory, picked for our setup (Windows, Claude Code only, heavy use of subagents and skills).

Upstream What it fixes Why we want it
obra#186 Test suite passes on Windows: .gitattributes forces LF, the parser derives the project name with path.dirname, and 2 POSIX-only tests are skipped Our fork had the split('/') parser bug, and core.autocrlf=true made every dist/ rebuild look modified
obra#168 Lines the harness injects (isMeta: system notifications, skill bodies, subagent messages) no longer split an exchange or get indexed as the user's words Our sessions are full of these, so search quality should improve
obra#172 better-sqlite3 12 → 13 (N-API binaries); postinstall verifies without rebuilding and runs a real vec0 query Survives Node version changes; 13.x has no install scripts
obra#179 The embedding model cache moves to ~/.config/superpowers/models (outside node_modules) Plugin updates no longer re-download the model

Skipped on purpose: obra#187 (secret redaction, +5k lines, gets its own review), obra#182 (mostly Codex), and the Codex/OMP-only PRs.

How it was applied

  • One squashed commit per PR, credited to its original author.
  • Upstream dist/ changes dropped: fix(embeddings): persist model cache outside node_modules obra/episodic-memory#179 carried a 6k-line bundle diff. dist/ was rebuilt here from source in a separate commit, which changes only the 6 files these PRs touch.
  • Safety-scanned before picking, and the verdict was SAFE: no hidden or bidi characters, no text aimed at an AI, no new network endpoints, no credential access. better-sqlite3 13.0.3 is the latest release from the same sole maintainer, with no install scripts.

Test plan

  • npm install → better-sqlite3 13.0.3, native stack verified
  • npm run build
  • npm test on Windows 11, Node 24.19: 369 passed, 8 skipped, 0 failed (65 files)
  • After merge: reinstall the plugin from the local marketplace so the cached plugin copy picks this up, then run one search to confirm

🤖 Generated with Claude Code

darthmolen and others added 5 commits October 10, 2026 03:17
From obra#186 (3 commits, squashed).
Upstream dist/ changes dropped; dist/ is rebuilt from source separately.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…es must not split an exchange (Claude isMeta, Codex wrappers)

From obra#168 (2 commits, squashed).
Upstream dist/ changes dropped; dist/ is rebuilt from source separately.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…exercise vector MCP path

From obra#172 (2 commits, squashed).
Upstream dist/ changes dropped; dist/ is rebuilt from source separately.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…utside node_modules

From obra#179 (4 commits, squashed).
Upstream dist/ changes dropped; dist/ is rebuilt from source separately.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@AndrewSpell
AndrewSpell merged commit 67ae499 into main Oct 10, 2026
2 checks passed
@AndrewSpell
AndrewSpell deleted the upstream-picks-186-168-172-179 branch October 10, 2026 08:35
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.

5 participants