refactor(buzz): remove legacy Sprout symlink paths - #154
Merged
Conversation
The Buzz desktop app rename from Sprout is fully shipped upstream, so the transitional `xyz.block.sprout.app` bundle paths are no longer needed.
wpfleger96
added a commit
that referenced
this pull request
Jul 10, 2026
#154 stopped creating the pre-rename xyz.block.sprout.app{,.dev} pack symlinks but left them on machines that installed earlier, and stale app state (e.g. Buzz teams.json source_dir) kept resolving through them. Surface them via the existing get_deprecated_symlinks cleanup hook so install/apply/uninstall remove them.
wpfleger96
added a commit
that referenced
this pull request
Jul 10, 2026
…#197) Two related cleanups to the Sietch Tabr pack deployment. **Persona runtime declarations.** Adds `runtime: buzz-agent` to the frontmatter of `paul.persona.md` and `duncan.persona.md`. Thufir and Alia already declare their runtimes (`codex` / `claude`), but Paul and Duncan resolved to `buzz-agent` only through the Buzz desktop app's default-runtime fallthrough. Declaring it in the pack makes the intent explicit and durable across team re-imports and persona materialization. **Legacy Sprout symlink removal.** #154 removed the pre-rename `xyz.block.sprout.app` / `xyz.block.sprout.app.dev` bundles from `BuzzTool.symlinks`, but machines that installed earlier still carry pack symlinks at those locations — and stale app state (e.g. the Buzz `teams.json` `source_dir`) keeps resolving through them. `BuzzTool` now surfaces those two paths via the existing `get_deprecated_symlinks()` hook, so `install`/`apply`/`uninstall` actively remove them. Removal only ever unlinks actual symlinks — regular files or directories at those paths are left untouched. Related: [block/buzz#1715](block/buzz#1715), which fixed how runtime and agent-command edits propagate from persona definitions to agent instances.
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
1 participant
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.
The Buzz desktop app rename from Sprout is fully shipped upstream, so the transitional
xyz.block.sprout.appbundle symlink paths are dead code and can be removed.BuzzTool.symlinkswas maintaining four symlink targets — prod and dev for bothxyz.block.buzz.appandxyz.block.sprout.app— so that persona packs would install into both app data directories during the transition period. Since the old Sprout bundle no longer exists, only the two Buzz paths are needed.get_legacy_sprout_teams_dir()fromplatform.pybuzz.pytest_buzz_tool_legacy_sprout_symlinksand update threelen(links)assertions from 4 → 2 intest_buzz_tool.pysprout-agenttobuzz-agentininstructions.md