Skip to content

🤖 fix(buzz): remove dead persona-pack target and clean up stale symlinks - #247

Merged
wpfleger96 merged 3 commits into
mainfrom
duncan/remove-buzz-pack-target
Sep 28, 2026
Merged

wpfleger96 merged 3 commits into
mainfrom
duncan/remove-buzz-pack-target

Conversation

@wpfleger96

@wpfleger96 wpfleger96 commented Sep 28, 2026 •

Copy link
Copy Markdown
Owner

Buzz stopped reading folder-based persona packs in block/buzz#1846. Since then, the BuzzTool symlink from config/buzz/ into …/agents/teams/com.wpfleger.sietch-tabr does nothing. This PR removes that dead target: src/ai_rules/config/buzz/, tools/buzz.py, platform.get_buzz_teams_dir, the registry entry, the tests and the doc tree lines.

It also adds a removal-only tombstone to SharedAgent.get_deprecated_symlinks(), so that the next ai-rules install deletes the stale links already on existing machines. SharedAgent hosts it because select_targets always includes the shared target, even with --agents filters. The pack id is hardcoded, since the plugin.json it used to be read from is deleted. The tombstone covers the Buzz prod and dev bundles plus the old Sprout ones, and keeps the per-OS app data base dirs. ConfigComponent.plan() now counts existing deprecated links as work, so the cleanup still runs when filters and exclude_symlinks leave no ordinary config symlinks to link.

The session-search Buzz relay reader is independent of the pack and stays unchanged.

Related: #248

BEGIN_COMMIT_OVERRIDE
fix(buzz): remove dead persona-pack target and clean up stale symlinks (#247)
END_COMMIT_OVERRIDE

Duncan and others added 3 commits September 28, 2026 17:56
Buzz stopped reading folder-based persona packs in block/buzz#1846, so the
BuzzTool symlink into agents/teams/ is inert. The tombstone on SharedAgent
(always selected) removes the leftover Buzz and Sprout pack links on install.

Co-authored-by: Will Pfleger <[email protected]>
Signed-off-by: Will Pfleger <[email protected]>
ConfigComponent.plan() counted only ordinary symlinks, so the runner skipped apply() — and the deprecated-symlink cleanup inside it — when filters and exclusions left nothing else to link. Planning now uses cleanup_deprecated_symlinks' predicate.

Co-authored-by: Will Pfleger <[email protected]>
Signed-off-by: Will Pfleger <[email protected]>
ConfigComponent discarded cleanup_deprecated_symlinks' return value, so cleanup-only runs listed each removal and then reported 'Summary: No changes'.

Co-authored-by: Will Pfleger <[email protected]>
Signed-off-by: Will Pfleger <[email protected]>
@wpfleger96
wpfleger96 merged commit 8b0d983 into main Sep 28, 2026
13 checks passed
@wpfleger96
wpfleger96 deleted the duncan/remove-buzz-pack-target branch September 28, 2026 22:30
wpfleger96 added a commit that referenced this pull request Sep 28, 2026
Buzz stopped reading folder-based persona packs in
[block/buzz#1846](block/buzz#1846), so the
`config/buzz` pack that #247 deletes can't be imported anymore. This
adds `backups/buzz/sietch-tabr.team.json`, an unmodified Buzz team
snapshot export (`buzz-team-snapshot` v1) that Buzz can import, plus a
README listing the per-agent effort and parallelism the export drops,
how to restore, and how to refresh.

`backups/` is at the repo top level, outside `src/`, so it isn't
included in the wheel or sdist.

Related: #247

Signed-off-by: Will Pfleger <[email protected]>
Co-authored-by: Duncan <dcfd242e557282d7a1e2cf2e6877522682f1e5c6156dc92ca7d90eaedd3b0f95@buzz.block.builderlab.xyz>
wpfleger96 pushed a commit that referenced this pull request Sep 29, 2026
🤖 I have created a release *beep* *boop*
---


##
[0.79.0](v0.78.0...v0.79.0)
(2026-09-29)


### Features

* **profiles:** add profile-owned skills
([#251](#251))
([8229776](8229776))
* **skills:** add writing-as-will voice skill
([#250](#250))
([8dc0b41](8dc0b41))


### Bug Fixes

* **buzz:** remove dead persona-pack target and clean up stale symlinks
([#247](#247))
([8b0d983](8b0d983))
* **install:** write through user symlinks and link from physical paths
([#254](#254))
([a83ebfa](a83ebfa))
* **uninstall:** remove only links that point at ai-rules sources
([#252](#252))
([a92cdd8](a92cdd8))


### Chores

* **buzz:** add Sietch Tabr team snapshot backup
([#248](#248))
([db34ae4](db34ae4))
* **buzz:** embed Paul's avatar in Sietch Tabr backup
([#249](#249))
([77eceb2](77eceb2))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

---------

Co-authored-by: wpfleger-release-bot[bot] <282999489+wpfleger-release-bot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
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