Skip to content

Move version.py into package dir; replace workflows with shared gh-automations callers#10

Merged
JarbasAl merged 5 commits into
devfrom
feat/core
Jun 13, 2026
Merged

Move version.py into package dir; replace workflows with shared gh-automations callers#10
JarbasAl merged 5 commits into
devfrom
feat/core

Conversation

@JarbasAl

Copy link
Copy Markdown
Contributor

Changes

  • version.py: moved from root to speakeronnx/version.py, updated pyproject.toml attr path
  • Workflows: replaced 2 inline workflows with 10 thin callers delegating to OpenVoiceOS/gh-automations

Workflows added

Workflow Purpose
build-tests.yml Matrix build + pytest via shared workflow
release_workflow.yml Alpha release on dev PR merge
publish_stable.yml Stable release on master push
release-preview.yml Predict next version on PRs
lint.yml Ruff linting
coverage.yml Code coverage
license_check.yml Dependency license audit
pip_audit.yml CVE scan
repo-health.yml Required files check + first-timer greeting
conventional-label.yml Auto-label PRs by conventional commit

@coderabbitai

coderabbitai Bot commented Jun 10, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@JarbasAl, we couldn't start this review because you've reached your PR review rate limit.

More reviews will be available in 17 minutes and 19 seconds. Learn how PR review limits work.

Your organization has run out of usage credits. Purchase more credits in the billing tab to continue.

⌛ How to resolve this issue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans include higher PR review limits than trial, open-source, and free plans. In all cases, reviews become available again over time. During sustained high-volume PR review activity, CodeRabbit may temporarily slow when the next review becomes available.

Please see our Fair Usage Limits Policy for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 8d84d0b5-89c2-42fc-a324-e8019f7acbd4

📥 Commits

Reviewing files that changed from the base of the PR and between 3a930bb and 7ce3fc5.

📒 Files selected for processing (15)
  • .github/workflows/build-tests.yml
  • .github/workflows/conventional-label.yml
  • .github/workflows/coverage.yml
  • .github/workflows/license_check.yml
  • .github/workflows/lint.yml
  • .github/workflows/pip_audit.yml
  • .github/workflows/publish.yml
  • .github/workflows/publish_stable.yml
  • .github/workflows/release-preview.yml
  • .github/workflows/release_workflow.yml
  • .github/workflows/repo-health.yml
  • .github/workflows/unit_tests.yml
  • pyproject.toml
  • speakeronnx/version.py
  • version.py
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/core

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

JarbasAl added 2 commits June 10, 2026 23:11
…, CLI

- SpeakerEmbedder with MODEL_REGISTRY (wespeaker-resnet34, wespeaker-ecapa512)
- 80-dim log-Mel fbank frontend + CMN in pure numpy (no librosa/soundfile)
- stdlib wave audio loader with soxr-optional resampling
- cosine() / verify() helpers
- speakeronnx CLI: embed, verify, list subcommands
- 19 unit tests (mocked session) + 8 e2e tests (real models, edge-tts audio)

e2e scores (en-US-GuyNeural vs en-US-JennyNeural, 3 clips each):
  wespeaker-resnet34: same=0.79/0.82, cross=0.15 — delta +0.64
  wespeaker-ecapa512: same=0.75/0.79, cross=0.11 — delta +0.64
@github-actions

github-actions Bot commented Jun 10, 2026

Copy link
Copy Markdown

Checking back in with the latest test results. 📡

I've aggregated the results of the automated checks for this PR below.

🏷️ Release Preview

Ensuring the release schedule is still on track. 🗓️

❌ Could not parse version.py: None

🔒 Security (pip-audit)

Security report: No threats detected in the area. ✅

✅ No known vulnerabilities found (50 packages scanned).

🔍 Lint

Checking if there's anything else we need to do. 📋

ruff: issues found — see job log

⚖️ License Check

Double-checking the fine print for any surprises. 🔍

❌ License violations detected (29 packages) — review required before merging.

Dependency                          License Name                                            License Type         Misc                                    
tqdm:4.68.2                         MPL-2.0 AND MIT                                         WeakCopyleft                                                 

License Type                        Found                                                  
WeakCopyleft                        1

License distribution: 7× MIT License, 5× MIT, 4× BSD-3-Clause, 3× Apache Software License, 1× 3-Clause BSD License, 1× Apache-2.0, 1× Apache-2.0 OR BSD-2-Clause, 1× BSD License, +6 more

Full breakdown — 29 packages
Package Version License URL
annotated-doc 0.0.4 MIT link
anyio 4.13.0 MIT link
build 1.5.0 MIT link
certifi 2026.5.20 Mozilla Public License 2.0 (MPL 2.0) link
click 8.4.1 BSD-3-Clause link
filelock 3.29.3 MIT link
flatbuffers 25.12.19 Apache Software License link
fsspec 2026.4.0 BSD-3-Clause link
h11 0.16.0 MIT License link
hf-xet 1.5.1 Apache-2.0 link
httpcore 1.0.9 BSD-3-Clause link
httpx 0.28.1 BSD License link
huggingface_hub 1.18.0 Apache Software License link
idna 3.18 BSD-3-Clause link
markdown-it-py 4.2.0 MIT License link
mdurl 0.1.2 MIT License link
numpy 2.4.6 BSD-3-Clause AND 0BSD AND MIT AND Zlib AND CC0-1.0 link
onnxruntime 1.26.0 MIT License link
packaging 26.2 Apache-2.0 OR BSD-2-Clause link
protobuf 7.35.0 3-Clause BSD License link
Pygments 2.20.0 BSD-2-Clause link
pyproject_hooks 1.2.0 MIT License link
PyYAML 6.0.3 MIT License link
rich 15.0.0 MIT License link
shellingham 1.5.4 ISC License (ISCL) link
speakeronnx 0.0.1a1 Apache Software License link
tqdm 4.68.2 MPL-2.0 AND MIT link
typer 0.25.1 MIT link
typing_extensions 4.15.0 PSF-2.0 link

Policy: Apache 2.0 (universal donor). StrongCopyleft / NetworkCopyleft / WeakCopyleft / Other / Error categories fail. MPL allowed.

🔨 Build Tests

The build report has been filed and is ready. 📁

✅ All versions pass

Python Build Install Tests
3.10
3.11
3.12
3.13
3.14

📋 Repo Health

Your repository is in great shape! ✅

⚠️ Some required files are missing.

Latest Version: 0.0.1a1

speakeronnx/version.py — Version file
README.md — README
LICENSE — License file
pyproject.toml — pyproject.toml
⚠️ setup.py — setup.py
⚠️ CHANGELOG.md — Changelog
speakeronnx/version.py has valid version block markers


Powered by OVOS scripts and a bit of magic. ✨

…tomations callers

- Move version.py from root to speakeronnx/version.py
- Update pyproject.toml attr path to speakeronnx.version.__version__
- Replace inline unit_tests.yml with shared build-tests.yml caller
- Split legacy publish.yml (master→alpha) into release_workflow.yml (dev→alpha) + publish_stable.yml (master→stable)
- Add lint, coverage, license_check, pip_audit, repo-health, release-preview, conventional-label workflows
JarbasAl and others added 2 commits June 11, 2026 00:08
test_all_hf_repos_reachable imports requests directly; it was absent
from the test extra so the test failed with ModuleNotFoundError.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
speakeronnx/version.py lacked START/END_VERSION_BLOCK markers required
by gh-automations release-preview; coverage.yml used non-existent 'dev'
extra instead of 'test'; license_check.yml now excludes the package
itself (pilosus does not parse the installed source build's license).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@JarbasAl JarbasAl marked this pull request as ready for review June 13, 2026 00:29
@JarbasAl JarbasAl merged commit 2720db5 into dev Jun 13, 2026
10 of 12 checks passed
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