Skip to content

fix(release): declare Debian package section - #24

Merged
rianjs merged 1 commit into
mainfrom
issue-18-deb-section
Aug 11, 2026
Merged

fix(release): declare Debian package section#24
rianjs merged 1 commit into
mainfrom
issue-18-deb-section

Conversation

@rianjs

@rianjs rianjs commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator

Fixes the package metadata defect found during #18 rollout.

  • declare Debian Section: sound at the Tauri bundle source
  • bump the corrective release to 0.2.1
  • keep Cargo/Tauri/lock versions aligned
  • make the release contract fail if the section disappears

Evidence: the v0.2.0 APT receiver downloaded both architectures, then reprepro rejected them with No section given for retune.

Checks:

  • node scripts/check-release.mjs
  • node scripts/check-docs.mjs
  • cargo fmt --all --check
  • cargo metadata --locked --no-deps
  • cargo test --workspace (318 passed, 1 hardware-only ignored)
  • desktop tests (23 passed)
  • git diff --check

Relates to #18.

@rianjs

rianjs commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator Author

Independent TDD/release assessment: no P1/P2/P3 findings. The committed release-contract check covers the regression at its source. Hosted acceptance still must inspect the built Debian control metadata and prove reprepro accepts both architectures before v0.2.1 is tagged.

@rianjs-bot rianjs-bot Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated PR Review

Reviewed commit: 5c0a9f11865f
Profile: codex-rianjs-bot - Posting as: rianjs-bot[bot]

Summary

Reviewer Findings
tauri:config-ipc 0
rust:implementation-tests 0
automation:ci-release 0
policies:conventions 0

Reviewer Coverage

  • tauri:config-ipc — complete (broad); inspected 1 assigned file (4 inspected across reviewers): apps/desktop/src-tauri/tauri.conf.json; skipped: none; constraints: Review limited to the assigned Tauri configuration change.
  • rust:implementation-tests — complete (broad); inspected 2 assigned files (4 inspected across reviewers): Cargo.lock, apps/desktop/src-tauri/Cargo.toml; skipped: none; constraints: Review limited to the assigned Cargo manifest and lockfile; release-contract/configuration files were outside this assignment. rtk cargo could not spawn Cargo in this environment, so metadata and desktop test execution could not be independently verified.
  • automation:ci-release — complete (broad); skipped: none; constraints: Cargo was unavailable in the review environment, so cargo metadata validation could not be run.
  • policies:conventions — complete (broad); skipped: none; constraints: Review limited to the assigned release/versioning metadata files and repository-local guidance; no shared CLI conventions checkout was available.
Inspected files (4)
  • Cargo.lock
  • apps/desktop/src-tauri/Cargo.toml
  • apps/desktop/src-tauri/tauri.conf.json
  • scripts/check-release.mjs

0 PR discussion threads considered. 0 summarized; 0 resolved.


Completed in 57s | gpt-5.6-terra | cr 0.10.284
Field Value
Model gpt-5.6-terra
Reviewers tauri:config-ipc, rust:implementation-tests, automation:ci-release, policies:conventions
Engine codex_cli · gpt-5.6-terra
Reviewed by cr · rianjs-bot[bot]
Duration 57s wall · 1m 49s compute
Cost unavailable
Tokens 291.2k in / 4.2k out

Per-workstream usage

  • orchestrator-selection — gpt-5.6-terra
    • In: 16.7k
    • Out: 339
    • Cache read: 11.0k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 9s
  • tauri:config-ipc — gpt-5.6-terra
    • In: 38.6k
    • Out: 398
    • Cache read: 29.2k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 12s
  • rust:implementation-tests — gpt-5.6-terra
    • In: 63.3k
    • Out: 942
    • Cache read: 50.4k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 25s
  • automation:ci-release — gpt-5.6-terra
    • In: 71.0k
    • Out: 1.2k
    • Cache read: 54.5k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 31s
  • policies:conventions — gpt-5.6-terra
    • In: 66.6k
    • Out: 971
    • Cache read: 51.5k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 25s
  • orchestrator-rollup — gpt-5.6-terra
    • In: 35.0k
    • Out: 380
    • Cache read: 27.1k
    • Cache create: unavailable
    • Cost: unavailable
    • Duration: 4s

@rianjs
rianjs marked this pull request as ready for review August 11, 2026 20:42
@rianjs

rianjs commented Aug 11, 2026

Copy link
Copy Markdown
Collaborator Author

Hosted release dry-run evidence from https://github.com/open-cli-collective/Retune/actions/runs/31534325377:

Both generated packages contain the expected fields:

retune_0.2.1_amd64.deb: Package retune; Version 0.2.1; Architecture amd64; Section sound; Priority optional
retune_0.2.1_arm64.deb: Package retune; Version 0.2.1; Architecture arm64; Section sound; Priority optional

The complete native matrix/aggregate is still running; merge remains gated on that run and normal CI.

@rianjs
rianjs merged commit fe2c479 into main Aug 11, 2026
24 checks passed
@rianjs
rianjs deleted the issue-18-deb-section branch August 11, 2026 21:05
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