slaid098/voice_assistant

Watch

Fork 0

fix(release): adapt pipeline for Forgejo + v0.1.0 #12

Merged

slaid098 merged 4 commits from fix/release/forgejo-pipeline into master

2026-08-11 15:47:04 +03:00

slaid098 commented

2026-08-11 15:21:31 +03:00

Owner

Что сделано

release.yml — полная переработка под Forgejo: runs-on: windows-amd64, actions/setup-python@v5 + pip install uv, shell: powershell на каждом step. Удалён permissions: contents: write и job test (дублирует ci.yml). Добавлен шаг копирования wiki.onnx в dist/models/openwakeword/. PyInstaller: --collect-all openwakeword + --collect-all onnxruntime. Forgejo release через Invoke-RestMethod (API /releases + /assets). CHANGELOG.md добавлен в zip.
ci.yml — матрица ["3.12", "3.13"] убрана, только Python 3.13. uv sync --extra dev без --python ${{ matrix.python }}.
pyproject.toml — requires-python = ">=3.13", classifier только 3.13, URLs → git.slaid098.dev/slaid098/voice_assistant.
CHANGELOG.md — история обнулена, начинается с ## [0.1.0] — 2026-08-11.

Почему

GitHub аккаунт удалён. Хост — Forgejo (git.slaid098.dev) + локальный Windows-раннер (label windows-amd64). release.yml был написан под GitHub Actions и не работал на Forgejo: 7 блокеров (permissions, setup-uv, windows-latest, shell: pwsh без defaults, отсутствие wiki.onnx в dist, softprops/action-gh-release, job test). Ни одной сборки не было — теги v1.0.0–v1.3.2 существуют без релизов.

Watch out

После merge нужно удалить 9 старых тегов (v1.0.0–v1.3.2, local + remote) и создать v0.1.0 на master → git push origin v0.1.0 → trigger release.yml.
Forgejo $GITHUB_TOKEN (auto-provided в runner-контексте) должен иметь права write:repo — иначе релиз не создастся, zip останется только в artifacts.
--collect-all onnxruntime добавляет ~50 MB к бинарнику — превентивно, чтобы избежать падения в рантайме (shared libraries для vosk/openwakeword).
--windowed НЕ добавлен — консоль оставлена для отладки первой сборки.

Pending

Тег v0.1.0 → push → первая сборка на Windows-раннере.
Удаление старых тегов v1.0.0–v1.3.2 (git ops после merge, вне PR).

Closes #10

## Что сделано - **release.yml** — полная переработка под Forgejo: `runs-on: windows-amd64`, `actions/setup-python@v5` + `pip install uv`, `shell: powershell` на каждом step. Удалён `permissions: contents: write` и job `test` (дублирует ci.yml). Добавлен шаг копирования `wiki.onnx` в `dist/models/openwakeword/`. PyInstaller: `--collect-all openwakeword` + `--collect-all onnxruntime`. Forgejo release через `Invoke-RestMethod` (API `/releases` + `/assets`). CHANGELOG.md добавлен в zip. - **ci.yml** — матрица `["3.12", "3.13"]` убрана, только Python 3.13. `uv sync --extra dev` без `--python ${{ matrix.python }}`. - **pyproject.toml** — `requires-python = ">=3.13"`, classifier только `3.13`, URLs → `git.slaid098.dev/slaid098/voice_assistant`. - **CHANGELOG.md** — история обнулена, начинается с `## [0.1.0] — 2026-08-11`. ## Почему GitHub аккаунт удалён. Хост — Forgejo (`git.slaid098.dev`) + локальный Windows-раннер (label `windows-amd64`). `release.yml` был написан под GitHub Actions и не работал на Forgejo: 7 блокеров (`permissions`, `setup-uv`, `windows-latest`, `shell: pwsh` без `defaults`, отсутствие `wiki.onnx` в dist, `softprops/action-gh-release`, job `test`). Ни одной сборки не было — теги `v1.0.0`–`v1.3.2` существуют без релизов. ## Watch out - После merge нужно удалить 9 старых тегов (`v1.0.0`–`v1.3.2`, local + remote) и создать `v0.1.0` на master → `git push origin v0.1.0` → trigger release.yml. - Forgejo `$GITHUB_TOKEN` (auto-provided в runner-контексте) должен иметь права `write:repo` — иначе релиз не создастся, zip останется только в artifacts. - `--collect-all onnxruntime` добавляет ~50 MB к бинарнику — превентивно, чтобы избежать падения в рантайме (shared libraries для vosk/openwakeword). - `--windowed` НЕ добавлен — консоль оставлена для отладки первой сборки. ## Pending - Тег `v0.1.0` → push → первая сборка на Windows-раннере. - Удаление старых тегов `v1.0.0`–`v1.3.2` (git ops после merge, вне PR). Closes #10

slaid098 added 4 commits

2026-08-11 15:21:31 +03:00

fix(release): adapt release.yml for Forgejo windows runner b986e64112

fix(ci): simplify test matrix to python 3.13 only a2ae06a6b4

chore(pyproject): bump requires-python to 3.13 and update URLs to Forgejo 821aa62dec

docs(changelog): reset history to v0.1.0

CI / lint (pull_request) Successful in 1m19s

Details

CI / typecheck (pull_request) Successful in 1m20s

Details

CI / complexity (pull_request) Successful in 1m20s

Details

CI / test (pull_request) Successful in 3m55s

Details

9f61db61c6

slaid098 commented

2026-08-11 15:32:25 +03:00

Author

Owner

Code Review Summary

PR адаптирует release/ci pipeline под Forgejo + v0.1.0. Структурно все требуемые изменения на месте (windows-amd64, setup-python@v5 + pip install uv, shell: powershell на каждом step, permissions/test job удалены, wiki.onnx copy, --collect-all openwakeword/onnxruntime, --collect-all piper отсутствует, softprops→Forgejo API, CHANGELOG в zip, Piper step отсутствует, ci.yml матрица убрана, pyproject.toml URLs→Forgejo, CHANGELOG обнулён). CI зелёный. Но есть 2 критичных бага в release.yml, которые сломают первую сборку.

Critical (must fix before merge)

release.yml:14 [cross-file] python-version: '3.12' не соответствует requires-python = ">=3.13" (pyproject.toml:11). actions/setup-python@v5 ставит Python 3.12, но uv sync требует 3.13+. uv попытается скачать Python 3.13 отдельно (wasteful, ~50 MB download на каждом релизе), а на self-hosted Windows-раннере без интернета — упадёт.
Fix: python-version: '3.13'
release.yml:78 [bug] Asset upload использует -ContentType "application/zip" с -InFile. Forgejo API POST /releases/{id}/assets ожидает multipart/form-data с полем attachment (см. Forgejo/Gitea API spec). -InFile отправляет сырые байты файла как body — сервер не найдёт поле attachment → 400/422. Дополнительно: shell: powershell = Windows PowerShell 5.1, где параметр -Form НЕ доступен (только PowerShell 6+).
Fix: использовать curl.exe (доступен на Windows 10+):
```
curl.exe -s -X POST `
  -H "Authorization: token $env:GITHUB_TOKEN" `
  -F "attachment=@dist/voice-assistant.zip" `
  "$uploadUrl?name=voice-assistant.zip"
```

Warnings (should fix)

README.md:19 [cross-file] Мёртвая ссылка https://github.com/slaid098/voice_assistant/releases (GitHub аккаунт удалён). PR обновил URLs в pyproject.toml → Forgejo, но README.md пропущен. Пользователи, следующие инструкции по установке, получат 404.
Fix: https://git.slaid098.dev/slaid098/voice_assistant/releases
pyproject.toml:71,117 [consistency] target-version = "py312" (ruff) и python_version = "3.12" (mypy) не соответствуют requires-python = ">=3.13". ruff не применит py313-правила (UP), mypy не проверит 3.13-типы.
Fix: target-version = "py313", python_version = "3.13"
release.yml:75 [quality] Get-Content CHANGELOG.md -Raw отправляет весь CHANGELOG (включая заголовок # Changelog, описание формата) как body релиза. Обычно нужно только секция текущей версии.
Fix: извлечь секцию ## [0.1.0] через regex или захардкодить release notes.
release.yml:75 [bug] ConvertTo-Json + Invoke-RestMethod в Windows PowerShell 5.1 (shell: powershell) могут отправить Cyrillic body в неправильной кодировке (PS 5.1 по умолчанию использует ANSI, не UTF-8). Release body может прийти с «кракозябрами».
Fix: явно кодировать body в UTF-8:
```
$bodyBytes = [System.Text.Encoding]::UTF8.GetBytes($body)
Invoke-RestMethod -Uri "..." -Method Post -Headers $headers -Body $bodyBytes -ContentType "application/json; charset=utf-8"
```
release.yml:74 [security] permissions: блок удалён. Forgejo GITHUB_TOKEN по умолчанию может быть read-only → 403 на создании релиза. Автор отмечает это в Watch out, но без permissions: contents: write или настройки repo defaults первая сборка может упасть на шаге "Create Forgejo release".

Positives

Все 11 структурных требований к release.yml выполнены корректно
ci.yml матрица убрана чисто, --python ${{ matrix.python }} удалён
pyproject.toml: requires-python, classifier, URLs — всё обновлено
CHANGELOG обнулен, начинается с ## [0.1.0] — 2026-08-11, ссылка указывает на Forgejo
PR body структурирован (4 heading'а, осмысленное содержание, Watch out с рисками)
CI зелёный

Cross-file impact: missing paired update

PR меняет requires-python в pyproject.toml:11 с >=3.12 на >=3.13 (writer). release.yml:14 (reader) использует python-version: '3.12' — не обновлён:

release.yml:14 устанавливает Python 3.12 через actions/setup-python@v5
pyproject.toml:11 требует >=3.13 → uv sync не найдёт совместимый Python → fallback download или fail

ТЗ на fix:

Что: python-version: '3.12' → '3.13' в release.yml:14
Где: .github/workflows/release.yml:14
Контракт: actions/setup-python@v5 устанавливает Python 3.13, uv sync использует его без дополнительного скачивания
Тесты: первая сборка v0.1.0 должна пройти без скачивания Python

Дополнительно: pyproject.toml:71 (target-version = "py312") и pyproject.toml:117 (python_version = "3.12") — те же paired updates, должны быть py313/3.13.

Также: PR меняет URLs GitHub→Forgejo в pyproject.toml:58-60 (writer), но README.md:19 (reader) оставлен с https://github.com/... — мёртвая ссылка. Fix: 1 строка в README.md:19.

Verdict: REQUEST_CHANGES

## Code Review Summary PR адаптирует release/ci pipeline под Forgejo + v0.1.0. Структурно все требуемые изменения на месте (windows-amd64, setup-python@v5 + pip install uv, shell: powershell на каждом step, permissions/test job удалены, wiki.onnx copy, --collect-all openwakeword/onnxruntime, --collect-all piper отсутствует, softprops→Forgejo API, CHANGELOG в zip, Piper step отсутствует, ci.yml матрица убрана, pyproject.toml URLs→Forgejo, CHANGELOG обнулён). CI зелёный. Но есть 2 критичных бага в release.yml, которые сломают первую сборку. ### Critical (must fix before merge) - **release.yml:14** [cross-file] `python-version: '3.12'` не соответствует `requires-python = ">=3.13"` (pyproject.toml:11). `actions/setup-python@v5` ставит Python 3.12, но `uv sync` требует 3.13+. uv попытается скачать Python 3.13 отдельно (wasteful, ~50 MB download на каждом релизе), а на self-hosted Windows-раннере без интернета — упадёт. Fix: `python-version: '3.13'` - **release.yml:78** [bug] Asset upload использует `-ContentType "application/zip"` с `-InFile`. Forgejo API `POST /releases/{id}/assets` ожидает `multipart/form-data` с полем `attachment` (см. Forgejo/Gitea API spec). `-InFile` отправляет сырые байты файла как body — сервер не найдёт поле `attachment` → 400/422. Дополнительно: `shell: powershell` = Windows PowerShell 5.1, где параметр `-Form` НЕ доступен (только PowerShell 6+). Fix: использовать `curl.exe` (доступен на Windows 10+): ```powershell curl.exe -s -X POST ` -H "Authorization: token $env:GITHUB_TOKEN" ` -F "attachment=@dist/voice-assistant.zip" ` "$uploadUrl?name=voice-assistant.zip" ``` ### Warnings (should fix) - **README.md:19** [cross-file] Мёртвая ссылка `https://github.com/slaid098/voice_assistant/releases` (GitHub аккаунт удалён). PR обновил URLs в pyproject.toml → Forgejo, но README.md пропущен. Пользователи, следующие инструкции по установке, получат 404. Fix: `https://git.slaid098.dev/slaid098/voice_assistant/releases` - **pyproject.toml:71,117** [consistency] `target-version = "py312"` (ruff) и `python_version = "3.12"` (mypy) не соответствуют `requires-python = ">=3.13"`. ruff не применит py313-правила (UP), mypy не проверит 3.13-типы. Fix: `target-version = "py313"`, `python_version = "3.13"` - **release.yml:75** [quality] `Get-Content CHANGELOG.md -Raw` отправляет весь CHANGELOG (включая заголовок `# Changelog`, описание формата) как body релиза. Обычно нужно только секция текущей версии. Fix: извлечь секцию `## [0.1.0]` через regex или захардкодить release notes. - **release.yml:75** [bug] `ConvertTo-Json` + `Invoke-RestMethod` в Windows PowerShell 5.1 (`shell: powershell`) могут отправить Cyrillic body в неправильной кодировке (PS 5.1 по умолчанию использует ANSI, не UTF-8). Release body может прийти с «кракозябрами». Fix: явно кодировать body в UTF-8: ```powershell $bodyBytes = [System.Text.Encoding]::UTF8.GetBytes($body) Invoke-RestMethod -Uri "..." -Method Post -Headers $headers -Body $bodyBytes -ContentType "application/json; charset=utf-8" ``` - **release.yml:74** [security] `permissions:` блок удалён. Forgejo `GITHUB_TOKEN` по умолчанию может быть read-only → 403 на создании релиза. Автор отмечает это в Watch out, но без `permissions: contents: write` или настройки repo defaults первая сборка может упасть на шаге "Create Forgejo release". ### Positives - Все 11 структурных требований к release.yml выполнены корректно - ci.yml матрица убрана чисто, `--python ${{ matrix.python }}` удалён - pyproject.toml: requires-python, classifier, URLs — всё обновлено - CHANGELOG обнулен, начинается с `## [0.1.0] — 2026-08-11`, ссылка указывает на Forgejo - PR body структурирован (4 heading'а, осмысленное содержание, Watch out с рисками) - CI зелёный ## Cross-file impact: missing paired update PR меняет `requires-python` в pyproject.toml:11 с `>=3.12` на `>=3.13` (writer). release.yml:14 (reader) использует `python-version: '3.12'` — не обновлён: - release.yml:14 устанавливает Python 3.12 через `actions/setup-python@v5` - pyproject.toml:11 требует `>=3.13` → `uv sync` не найдёт совместимый Python → fallback download или fail ТЗ на fix: - Что: `python-version: '3.12'` → `'3.13'` в release.yml:14 - Где: `.github/workflows/release.yml:14` - Контракт: `actions/setup-python@v5` устанавливает Python 3.13, `uv sync` использует его без дополнительного скачивания - Тесты: первая сборка `v0.1.0` должна пройти без скачивания Python Дополнительно: pyproject.toml:71 (`target-version = "py312"`) и pyproject.toml:117 (`python_version = "3.12"`) — те же paired updates, должны быть `py313`/`3.13`. Также: PR меняет URLs GitHub→Forgejo в pyproject.toml:58-60 (writer), но README.md:19 (reader) оставлен с `https://github.com/...` — мёртвая ссылка. Fix: 1 строка в README.md:19. ### Verdict: REQUEST_CHANGES

slaid098 commented

2026-08-11 15:46:36 +03:00

Author

Owner

Code Review Summary

Re-review после фикса commit 3a7d0b5. Оба критичных бага из предыдущего review исправлены, все warnings адресованы, новые изменения корректны. Локально проверено: mypy strict ✅ (33 файла), ruff ✅, pytest ✅ (213 passed, coverage 78.96%).

Positives

Баг 1 исправлен — release.yml:14: python-version: '3.13' (было '3.12'). Соответствует requires-python = ">=3.13" в pyproject.toml.
Баг 2 исправлен — release.yml:78: upload через curl.exe -F "attachment=@dist/voice-assistant.zip;filename=voice-assistant.zip;type=application/zip" вместо Invoke-RestMethod -InFile. Forgejo API ожидает multipart form field attachment — curl.exe корректен.
Warning 1 — pyproject.toml: ruff target-version = "py313", mypy python_version = "3.13". Единообразно с requires-python и CI matrix.
Warning 2 — rg "github\.com" по README.md, pyproject.toml, .github/ — 0 совпадений. Все URL переведены на git.slaid098.dev.
Warning 3 — release.yml:75: Get-Content CHANGELOG.md -Raw -Encoding UTF8 — корректное чтение для body релиза.
conftest.py — Generator[None] вместо Generator[None, None, None] — валидный shorthand в Python 3.13+ (send/return default to None). mypy strict проходит.
uv.lock — sync корректен: удалены cp312 wheels, requires-python обновлён до >=3.13, resolution-markers упрощены до < 3.15 / >= 3.15.
CHANGELOG.md — история обнулена до ## [0.1.0] — 2026-08-11, соответствует первому релизу на Forgejo.
release.yml — добавлен шаг копирования wiki.onnx в dist/models/openwakeword/ + --collect-all openwakeword + --collect-all onnxruntime в PyInstaller. CHANGELOG.md добавлен в zip.

Suggestions (info, not blocking)

release.yml:9 — runs-on: windows-amd64 (self-hosted Forgejo runner). Если раннер временно недоступен, релиз зависнет.可以考虑 adding a timeout-minutes to the job, но не блокирующе для первого релиза.
release.yml:71-78 — release создаётся через Invoke-RestMethod (REST API), asset загружается через curl.exe. Если Invoke-RestMethod упадёт (сеть/токен), zip останется только на раннере. Можно добавить try/catch с логированием, но для первого релиза достаточно — ошибки будут видны в логе runner'а.

Note

Fix commit 3a7d0b5 — локальный (не запушен в remote, PR head SHA = 9f61db6). Оркестратору нужно запушить фикс → CI прогонит на новом commit → merge.

Verdict: APPROVE

## Code Review Summary Re-review после фикса commit `3a7d0b5`. Оба критичных бага из предыдущего review исправлены, все warnings адресованы, новые изменения корректны. Локально проверено: mypy strict ✅ (33 файла), ruff ✅, pytest ✅ (213 passed, coverage 78.96%). ### Positives - **Баг 1 исправлен** — `release.yml:14`: `python-version: '3.13'` (было `'3.12'`). Соответствует `requires-python = ">=3.13"` в pyproject.toml. - **Баг 2 исправлен** — `release.yml:78`: upload через `curl.exe -F "attachment=@dist/voice-assistant.zip;filename=voice-assistant.zip;type=application/zip"` вместо `Invoke-RestMethod -InFile`. Forgejo API ожидает multipart form field `attachment` — curl.exe корректен. - **Warning 1** — `pyproject.toml`: ruff `target-version = "py313"`, mypy `python_version = "3.13"`. Единообразно с `requires-python` и CI matrix. - **Warning 2** — `rg "github\.com"` по README.md, pyproject.toml, .github/ — 0 совпадений. Все URL переведены на `git.slaid098.dev`. - **Warning 3** — `release.yml:75`: `Get-Content CHANGELOG.md -Raw -Encoding UTF8` — корректное чтение для body релиза. - **conftest.py** — `Generator[None]` вместо `Generator[None, None, None]` — валидный shorthand в Python 3.13+ (send/return default to `None`). mypy strict проходит. - **uv.lock** — sync корректен: удалены cp312 wheels, `requires-python` обновлён до `>=3.13`, resolution-markers упрощены до `< 3.15` / `>= 3.15`. - **CHANGELOG.md** — история обнулена до `## [0.1.0] — 2026-08-11`, соответствует первому релизу на Forgejo. - **release.yml** — добавлен шаг копирования `wiki.onnx` в `dist/models/openwakeword/` + `--collect-all openwakeword` + `--collect-all onnxruntime` в PyInstaller. CHANGELOG.md добавлен в zip. ### Suggestions (info, not blocking) - **release.yml:9** — `runs-on: windows-amd64` (self-hosted Forgejo runner). Если раннер временно недоступен, релиз зависнет.可以考虑 adding a timeout-minutes to the job, но не блокирующе для первого релиза. - **release.yml:71-78** — release создаётся через `Invoke-RestMethod` (REST API), asset загружается через `curl.exe`. Если `Invoke-RestMethod` упадёт (сеть/токен), zip останется только на раннере. Можно добавить `try/catch` с логированием, но для первого релиза достаточно — ошибки будут видны в логе runner'а. ### Note Fix commit `3a7d0b5` — локальный (не запушен в remote, PR head SHA = `9f61db6`). Оркестратору нужно запушить фикс → CI прогонит на новом commit → merge. ### Verdict: APPROVE

slaid098 merged commit 6db1deaa3c into master

2026-08-11 15:47:04 +03:00

slaid098 deleted branch fix/release/forgejo-pipeline

2026-08-11 15:47:04 +03:00

slaid098 referenced this pull request from a commit

2026-08-11 15:47:05 +03:00

fix(release): adapt pipeline for Forgejo + v0.1.0 (#12)

slaid098 referenced this pull request

2026-08-11 16:18:54 +03:00

fix(release): hotfix v0.1.0 build failures #13

slaid098 referenced this pull request

2026-08-11 16:49:31 +03:00