Skip to content

docs(opds): trim validation docs and fix Linux CI bundling - #155

Draft
phildenhoff wants to merge 1 commit into
opds-10-hardeningfrom
opds-11-packaging
Draft

phildenhoff wants to merge 1 commit into
opds-10-hardeningfrom
opds-11-packaging

Conversation

@phildenhoff

@phildenhoff phildenhoff commented Sep 18, 2026

Copy link
Copy Markdown
Member

Why

Packaged validation for this round is done, and the docs it produced read like a dated evidence log rather than anything a reader can act on. Separately, the Linux bundle turned out to fail in CI outright.

What changed

  • CI (build.yml + release.yml) installs xdg-utils on Ubuntu: the Linux bundler needs /usr/bin/xdg-open, and the packaged bundle failed without it — discovered during packaged validation, not by reading bundler docs.
  • New libcalibre example create_opds_validation_library generates a representative library — 105 books with Unicode and XML-metacharacter titles, mixed languages, missing covers, an 8 MiB file, a fileless book, genres — so validation libraries are reproducible instead of hand-built.
  • Deletes docs/opds-validation.md on purpose: it was a one-time record, mostly "Not run" rows, and the durable evidence lives in CI runs and release artifacts. Trims docs/opds-sharing.md to setup only — how to turn on sharing, and that any OPDS 1.x reader works.
  • src-tauri/Cargo.toml now uses rust-version.workspace = true instead of its own pinned copy.

Validation: the stack tip runs 62 citadel-opds tests + 348 workspace tests, all green.

Stack: #146#147#148#149#150#151#152#153#154#155 (this, top)

@github-actions

Copy link
Copy Markdown

libcalibre Test Coverage Report

Overall coverage: 83.86%

📊 Download HTML Report

Coverage breakdown available in the artifacts.

@github-actions

Copy link
Copy Markdown

libcalibre Test Coverage Report

Overall coverage: 83.86%

📊 Download HTML Report

Coverage breakdown available in the artifacts.

@github-actions

Copy link
Copy Markdown

libcalibre Test Coverage Report

Overall coverage: 83.86%

📊 Download HTML Report

Coverage breakdown available in the artifacts.

@github-actions

Copy link
Copy Markdown

libcalibre Test Coverage Report

Overall coverage: 83.86%

📊 Download HTML Report

Coverage breakdown available in the artifacts.

@github-actions

Copy link
Copy Markdown

libcalibre Test Coverage Report

Overall coverage: 84.01%

📊 Download HTML Report

Coverage breakdown available in the artifacts.

@github-actions

Copy link
Copy Markdown

libcalibre Test Coverage Report

Overall coverage: 84.01%

📊 Download HTML Report

Coverage breakdown available in the artifacts.

@github-actions

Copy link
Copy Markdown

libcalibre Test Coverage Report

Overall coverage: 84.01%

📊 Download HTML Report

Coverage breakdown available in the artifacts.

@github-actions

Copy link
Copy Markdown

libcalibre Test Coverage Report

Overall coverage: 84.01%

📊 Download HTML Report

Coverage breakdown available in the artifacts.

@github-actions

Copy link
Copy Markdown

libcalibre Test Coverage Report

Overall coverage: 84.01%

📊 Download HTML Report

Coverage breakdown available in the artifacts.

@github-actions

Copy link
Copy Markdown

libcalibre Test Coverage Report

Overall coverage: 84.01%

📊 Download HTML Report

Coverage breakdown available in the artifacts.

@github-actions

Copy link
Copy Markdown

libcalibre Test Coverage Report

Overall coverage: 84.01%

📊 Download HTML Report

Coverage breakdown available in the artifacts.

test(opds): add representative package fixture

test(opds): validate Linux packages

docs(opds): verify signed nightly artifact

docs(opds): drop stale commit ids from validation record

Commit ids change on every rebase; the immutable evidence is the release
tag and the artifact digest, which stay.
@github-actions

Copy link
Copy Markdown

libcalibre Test Coverage Report

Overall coverage: 84.01%

📊 Download HTML Report

Coverage breakdown available in the artifacts.

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