This repository was archived by the owner on Sep 11, 2026. It is now read-only.
Repository navigation
feat(taze): cross-repo migration cache + signal-safe worktree cleanup - #152
Merged
Merged
Conversation
Running n-taze across sibling repos that share the same major dependency bump (e.g. mlmail/myllm/myshare all on @7n/tauri-components ^0.8.0) made the runner rediscover the same breaking-change analysis from scratch in every repo. readMigrationCache/writeMigrationCache key on (pkg, from, to) so the first repo's analysis is reused as prompt context by the rest. Also: ensureRunningInWorktree's auto-created worktree was only cleaned up in the try/finally — a SIGINT/SIGTERM (Ctrl-C, an external timeout) skipped it entirely and left an orphaned worktree with partial edits. Reproduced this live during a taze run on vitaliytv/myllm. Added a signal handler that runs the same bring-changes-back + worktree-remove path before exiting. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
3 tasks done
vitaliytv
added a commit
that referenced
this pull request
Jul 19, 2026
#153) Follow-up to #152 — the merge exposed 6 delta-lint violations in the new test mocks (oxlint require-await on async stubs with no await inside, no-empty-function on a bare async () => {}). Also hardens the new SIGTERM test against the ensureRunningInWorktree async-ification from #151: the signal handler now registers after a microtask hop, so the test polls for it instead of asserting synchronously right after the call. Co-authored-by: Claude Sonnet 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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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.
Summary
migration-cache.mjs) keyed on(pkg, from, to): when sibling repos share the same major dependency bump (e.g.mlmail/myllm/myshareall pinned to@7n/tauri-components ^0.8.0), only the first repo'sn-tazerun does the full CHANGELOG/breaking-change investigation — the rest get that summary prepended to their prompt instead of rediscovering it from scratch.runTazeOrchestrator. Previously an interrupted process (Ctrl-C, an external timeout) skipped thefinallycleanup entirely and left an orphaned.worktrees/<branch>-tazewith partial edits — reproduced live during atazerun onvitaliytv/myllm. The signal handler now runs the same bring-changes-back + worktree-remove path before exiting.Test plan
npx vitest run npm/skills/taze/js/tests/orchestrate.test.mjs npm/skills/taze/js/tests/migration-cache.test.mjs— 278 passed, including 10 new tests (cache hit/miss, cache write-after-success, SIGTERM cleanup)npx vitest run(full repo suite) — all greennpx eslinton all touched files — clean🤖 Generated with Claude Code