feat: rename package and CLI to SynthBanshee#3
Merged
Conversation
Outward-facing name is now SynthBanshee throughout. AVDP acronym is retained in doc context (it is the parent initiative) but not as the tool name. Changes: - avdp/ → synthbanshee/ (Python package directory, git mv) - All `from avdp.*` / `import avdp` → synthbanshee in package + tests - pyproject.toml: name avdp-synth → synthbanshee, entry point avdp → synthbanshee, hatch packages, coverage source/paths - .pre-commit-config.yaml: mypy files pattern avdp/ → synthbanshee/ - .github/workflows/ci.yml: --cov and mypy targets updated - CLAUDE.md / AGENTS.md: all synthbanshee/ path references updated - README.md: full rewrite — goal, pipeline stages, tier table, taxonomy summary, current status, quick-start, and docs table Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This comment has been minimized.
This comment has been minimized.
There was a problem hiding this comment.
Pull request overview
Renames the project’s public identity from avdp to SynthBanshee across the Python package, CLI entrypoint, and project metadata, aligning repository docs/tooling with the new name while keeping “AVDP” as the parent initiative context.
Changes:
- Renamed import root throughout tests and modules to
synthbanshee.*and updated CLI script entrypoint tosynthbanshee. - Added/moved core modules under
synthbanshee/(TTS, preprocessing, config/taxonomy) and updated docs to reflect the new branding. - Updated tooling configuration (coverage, mypy target, pre-commit file filter, CI coverage target) to point at
synthbanshee.
Reviewed changes
Copilot reviewed 24 out of 31 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| tests/unit/test_tts.py | Updates unit tests to import TTS components from synthbanshee. |
| tests/unit/test_preprocessing.py | Updates preprocessing tests to import from synthbanshee. |
| tests/unit/test_labels.py | Updates label tests to import from synthbanshee. |
| tests/unit/test_config.py | Updates config tests to import from synthbanshee. |
| tests/integration/test_happy_path.py | Updates integration test imports to synthbanshee modules. |
| synthbanshee/tts/ssml_builder.py | Adds SSML builder implementation under the renamed package. |
| synthbanshee/tts/renderer.py | Updates renderer to use synthbanshee imports. |
| synthbanshee/tts/azure_provider.py | Adds Azure TTS provider implementation under the renamed package. |
| synthbanshee/tts/init.py | Re-exports TTS APIs from the new package namespace. |
| synthbanshee/script/templates/she_proves/stub_utterance.txt | Adds a stub utterance template used by the CLI’s phase-0 generation path. |
| synthbanshee/package/validator.py | Updates validator imports to synthbanshee. |
| synthbanshee/package/init.py | Re-exports validator API from synthbanshee. |
| synthbanshee/labels/schema.py | Updates label schema to import taxonomy from synthbanshee. |
| synthbanshee/labels/generator.py | Updates generator to use synthbanshee.__version__ and schema imports. |
| synthbanshee/labels/init.py | Re-exports label schema types from synthbanshee. |
| synthbanshee/config/taxonomy.py | Adds taxonomy loader under the new package. |
| synthbanshee/config/speaker_config.py | Updates taxonomy import to synthbanshee. |
| synthbanshee/config/scene_config.py | Updates config imports to synthbanshee. |
| synthbanshee/config/acoustic_config.py | Adds acoustic config models under the new package. |
| synthbanshee/config/init.py | Adds config package exports under synthbanshee. |
| synthbanshee/cli.py | Updates CLI imports to synthbanshee to support the renamed entrypoint. |
| synthbanshee/augment/preprocessing.py | Adds preprocessing pipeline implementation under the new package. |
| synthbanshee/augment/init.py | Updates augment exports to synthbanshee. |
| synthbanshee/init.py | Adds package root and version under synthbanshee. |
| README.md | Rewrites README to describe SynthBanshee purpose, pipeline, taxonomy, and quick start. |
| pyproject.toml | Renames PyPI package and CLI script entrypoint; updates coverage source and wheel packages. |
| CLAUDE.md | Updates repo structure and references to synthbanshee naming. |
| avdp/config/init.py | Removes old avdp config package exports (part of rename cleanup). |
| AGENTS.md | Updates repo structure and references to synthbanshee naming. |
| .pre-commit-config.yaml | Updates mypy pre-commit file filter to ^synthbanshee/. |
| .github/workflows/ci.yml | Updates CI coverage and mypy target paths to synthbanshee. |
Comments suppressed due to low confidence (1)
synthbanshee/cli.py:96
- The CLI still hardcodes the old
avdppaths/naming in a couple places: the module docstring usage examples referenceavdp ..., andstub_templateis loaded fromavdp/script/templates/...even though the file now lives undersynthbanshee/script/.... This makes the template lookup fail (falling back to the placeholder text) and is confusing post-rename. Update those strings/paths to the newsynthbansheelocations.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
This comment has been minimized.
This comment has been minimized.
- Fix leftover avdp/ path reference in stub_template lookup (line 121
was still pointing at avdp/script/templates/ after the rename)
- Fix module docstring usage examples (avdp → synthbanshee)
- Add tests/unit/test_cli.py covering:
- generate --dry-run (covers line 69: SceneConfig import path)
- generate --dry-run with missing config (non-zero exit)
- generate full pipeline with mocked TTSRenderer (covers lines 92-97:
lazy imports in non-dry-run branch)
- validate on a valid clip (covers line 223: validator import path)
- validate on an invalid clip (non-zero exit)
Patch coverage for the 8 originally flagged lines (69, 92-97, 223) is
now 100%. COPILOT-1 (wheel data-file packaging) tracked in issue #4
and resolved as out-of-scope for this rename PR.
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
pr-agent-context report: 🚨 `pr-agent-context` failed while preparing PR context.
PR: #3
Error: CalledProcessError: Command '['git', '-C', '/home/runner/work/SynthBanshee/SynthBanshee/caller-repo', 'diff', '--unified=0', '0e69ee8b656eef0832031037ad6484fdfc639f52...03140f7faf64955afee058bab1e330fe02f95cb8']' returned non-zero exit status 128.
Run: https://github.com/DataHackIL/SynthBanshee/actions/runs/24080603462
The workflow continued gracefully so this failure does not block CI.
Check the job logs for the full traceback.Run metadata: |
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
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
avdp/→synthbanshee/(all imports updated)avdp→synthbanshee(i.e. `synthbanshee generate`, `synthbanshee validate`)avdp-synth→synthbansheeAVDP acronym is retained in documentation context (it is the parent initiative) but is no longer the tool name anywhere outward-facing.
What changed
avdp/synthbanshee/avdp generatesynthbanshee generateavdp-synthsynthbansheefrom avdp.*from synthbanshee.*avdpsynthbansheeavdp/synthbanshee/Test plan
pytest tests/)synthbanshee --helpworks after install🤖 Generated with Claude Code