Skip to content

fix(api): preserve media type for scoped book lookup - #114

Open
snapetech wants to merge 1416 commits into
Chaptarr:developfrom
snapetech-org:main
Open

snapetech wants to merge 1416 commits into
Chaptarr:developfrom
snapetech-org:main

Conversation

@snapetech

Copy link
Copy Markdown

Fixes #67

Book lookup already accepts a mediaType query parameter and filters remote results with it, but the remote text-search path was not given that value. Goodreads autocomplete results therefore kept the default Audiobook media type, so an ebook lookup filtered every result out.

This passes the requested BookMediaType from BookLookupController through ISearchForNewBook and BookInfoProxy. Calls without a scope keep the existing behavior.

I added regression coverage for both the controller forwarding the requested type and Goodreads text results retaining it.

Testing:

  • dotnet build src/Chaptarr.sln --configuration Release --no-restore
  • dotnet test src/Chaptarr.Core.Test/Chaptarr.Core.Test.csproj --configuration Release --no-restore
  • 2,854 tests passed

Signed-off-by: snapetech <keith@snape.tech>
Signed-off-by: snapetech <keith@snape.tech>
Back up each BookshelfNG PostgreSQL database with pg_dump before migration, keep the source idle through review and apply, and restore validated dumps before Windows rollback. Add a real-tool disposable PostgreSQL fixture to the hosted migration job.

Also provide synthetic passwd/group files in the trusted Linux test sandbox so NUnit can resolve its isolated identity, and make mocked Hardcover image tests independent of network DNS.
The validation-engine removal replaced receipt upload with direct dotnet test execution. Assert that the current test step runs before the frontend build and publish steps.

release-note: none
snapetech and others added 30 commits October 26, 2026 14:15
Bumps [azure/setup-helm](https://github.com/azure/setup-helm) from 4.3.1 to 5.0.1.
- [Release notes](https://github.com/azure/setup-helm/releases)
- [Changelog](https://github.com/Azure/setup-helm/blob/main/CHANGELOG.md)
- [Commits](Azure/setup-helm@1a275c3...9bc31f4)

---
updated-dependencies:
- dependency-name: azure/setup-helm
  dependency-version: 5.0.1
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit 90177f2)
Bumps [aquasecurity/setup-trivy](https://github.com/aquasecurity/setup-trivy) from e07451d2e059ed86c2870430ea286b3a9e0bf241 to 81e514348e19b6112ce2a7e3ecbafe19c1e1f567.
- [Release notes](https://github.com/aquasecurity/setup-trivy/releases)
- [Commits](aquasecurity/setup-trivy@e07451d...81e5143)

---
updated-dependencies:
- dependency-name: aquasecurity/setup-trivy
  dependency-version: 81e514348e19b6112ce2a7e3ecbafe19c1e1f567
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit afb580d)
Bumps the actions-minor-patch group with 3 updates: [ruby/setup-ruby](https://github.com/ruby/setup-ruby), [taiki-e/install-action](https://github.com/taiki-e/install-action) and [github/codeql-action/upload-sarif](https://github.com/github/codeql-action).

Updates `ruby/setup-ruby` from 1.321.0 to 1.331.0
- [Release notes](https://github.com/ruby/setup-ruby/releases)
- [Changelog](https://github.com/ruby/setup-ruby/blob/master/release.rb)
- [Commits](ruby/setup-ruby@95ef2b0...824c0e2)

Updates `taiki-e/install-action` from 2.87.28 to 2.87.29
- [Release notes](https://github.com/taiki-e/install-action/releases)
- [Changelog](https://github.com/taiki-e/install-action/blob/main/CHANGELOG.md)
- [Commits](taiki-e/install-action@1168f68...8cca091)

Updates `github/codeql-action/upload-sarif` from 4.38.3 to 4.39.0
- [Release notes](https://github.com/github/codeql-action/releases)
- [Changelog](https://github.com/github/codeql-action/blob/main/CHANGELOG.md)
- [Commits](github/codeql-action@v4.38.3...6f2ac81)

---
updated-dependencies:
- dependency-name: ruby/setup-ruby
  dependency-version: 1.331.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: actions-minor-patch
- dependency-name: taiki-e/install-action
  dependency-version: 2.87.29
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: actions-minor-patch
- dependency-name: github/codeql-action/upload-sarif
  dependency-version: 4.39.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: actions-minor-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit 4b5144f)
Bumps the npm-minor-patch group with 4 updates: [qs](https://github.com/ljharb/qs), [core-js](https://github.com/zloirock/core-js/tree/HEAD/packages/core-js), [postcss](https://github.com/postcss/postcss) and [webpack](https://github.com/webpack/webpack).

Updates `qs` from 6.16.0 to 6.16.1
- [Changelog](https://github.com/ljharb/qs/blob/main/CHANGELOG.md)
- [Commits](ljharb/qs@v6.16.0...v6.16.1)

Updates `core-js` from 3.50.0 to 3.51.0
- [Release notes](https://github.com/zloirock/core-js/releases)
- [Changelog](https://github.com/zloirock/core-js/blob/master/CHANGELOG.md)
- [Commits](https://github.com/zloirock/core-js/commits/v3.51.0/packages/core-js)

Updates `postcss` from 8.5.30 to 8.5.31
- [Release notes](https://github.com/postcss/postcss/releases)
- [Changelog](https://github.com/postcss/postcss/blob/main/CHANGELOG.md)
- [Commits](postcss/postcss@8.5.30...8.5.31)

Updates `webpack` from 5.111.1 to 5.112.0
- [Release notes](https://github.com/webpack/webpack/releases)
- [Changelog](https://github.com/webpack/webpack/blob/main/CHANGELOG.md)
- [Commits](webpack/webpack@v5.111.1...v5.112.0)

---
updated-dependencies:
- dependency-name: qs
  dependency-version: 6.16.1
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: npm-minor-patch
- dependency-name: core-js
  dependency-version: 3.51.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: npm-minor-patch
- dependency-name: postcss
  dependency-version: 8.5.31
  dependency-type: direct:development
  update-type: version-update:semver-patch
  dependency-group: npm-minor-patch
- dependency-name: webpack
  dependency-version: 5.112.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: npm-minor-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit 0dc73ff)
Bumps [@babel/core](https://github.com/babel/babel/tree/HEAD/packages/babel-core) from 7.29.6 to 7.29.7.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.29.7/packages/babel-core)

---
updated-dependencies:
- dependency-name: "@babel/core"
  dependency-version: 7.29.7
  dependency-type: direct:development
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit aaa44f5)
Bumps [@babel/preset-typescript](https://github.com/babel/babel/tree/HEAD/packages/babel-preset-typescript) from 7.26.0 to 7.29.7.
- [Release notes](https://github.com/babel/babel/releases)
- [Changelog](https://github.com/babel/babel/blob/main/CHANGELOG.md)
- [Commits](https://github.com/babel/babel/commits/v7.29.7/packages/babel-preset-typescript)

---
updated-dependencies:
- dependency-name: "@babel/preset-typescript"
  dependency-version: 7.29.7
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit f9e58f8)
Bumps [postcss-nested](https://github.com/postcss/postcss-nested) from 6.2.0 to 9.0.0.
- [Release notes](https://github.com/postcss/postcss-nested/releases)
- [Changelog](https://github.com/postcss/postcss-nested/blob/main/CHANGELOG.md)
- [Commits](postcss/postcss-nested@6.2.0...9.0.0)

---
updated-dependencies:
- dependency-name: postcss-nested
  dependency-version: 9.0.0
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit b805f84)
Bumps [react-dnd-multi-backend](https://github.com/LouisBrunner/dnd-multi-backend/tree/HEAD/packages/react-dnd-multi-backend) from 8.0.1 to 9.0.0.
- [Release notes](https://github.com/LouisBrunner/dnd-multi-backend/releases)
- [Commits](https://github.com/LouisBrunner/dnd-multi-backend/commits/v9.0.0/packages/react-dnd-multi-backend)

---
updated-dependencies:
- dependency-name: react-dnd-multi-backend
  dependency-version: 9.0.0
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit 8a6eca0)
Bumps [redux-actions](https://github.com/redux-utilities/redux-actions) from 2.6.5 to 3.0.3.
- [Release notes](https://github.com/redux-utilities/redux-actions/releases)
- [Changelog](https://github.com/redux-utilities/redux-actions/blob/master/docs/Changelog.md)
- [Commits](redux-utilities/redux-actions@v2.6.5...v3.0.3)

---
updated-dependencies:
- dependency-name: redux-actions
  dependency-version: 3.0.3
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit 7afba02)
Bumps [stylelint-order](https://github.com/hudochenkov/stylelint-order) from 6.0.4 to 8.1.2.
- [Release notes](https://github.com/hudochenkov/stylelint-order/releases)
- [Changelog](https://github.com/hudochenkov/stylelint-order/blob/master/CHANGELOG.md)
- [Commits](hudochenkov/stylelint-order@6.0.4...8.1.2)

---
updated-dependencies:
- dependency-name: stylelint-order
  dependency-version: 8.1.2
  dependency-type: direct:development
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit c9fec4d)
Bumps dotnet/aspnet from 10.0.13-noble to 10.0.14-noble.

---
updated-dependencies:
- dependency-name: dotnet/aspnet
  dependency-version: 10.0.14-noble
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit 43a864c)
Bumps dotnet/sdk from 10.0.402 to 10.0.403.

---
updated-dependencies:
- dependency-name: dotnet/sdk
  dependency-version: 10.0.403
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit 7c130c6)
Bumps NLog from 6.2.1 to 6.2.2
Bumps NLog.Extensions.Logging from 6.2.1 to 6.2.2
Bumps PDFsharp from 6.2.4 to 6.5.0
Bumps Swashbuckle.AspNetCore.Annotations from 10.3.0 to 10.3.1
Bumps Swashbuckle.AspNetCore.SwaggerGen from 10.3.0 to 10.3.1

---
updated-dependencies:
- dependency-name: NLog
  dependency-version: 6.2.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-minor-patch
- dependency-name: NLog.Extensions.Logging
  dependency-version: 6.2.2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-minor-patch
- dependency-name: PDFsharp
  dependency-version: 6.5.0
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: nuget-minor-patch
- dependency-name: Swashbuckle.AspNetCore.Annotations
  dependency-version: 10.3.1
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-minor-patch
- dependency-name: Swashbuckle.AspNetCore.SwaggerGen
  dependency-version: 10.3.1
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: nuget-minor-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
(cherry picked from commit b60d10c)
…th 2 updates

Bumps the npm-minor-patch group with 2 updates in the / directory: [@typescript-eslint/eslint-plugin](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/eslint-plugin) and [@typescript-eslint/parser](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/parser).


Updates `@typescript-eslint/eslint-plugin` from 8.71.2 to 8.72.0
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/eslint-plugin/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.72.0/packages/eslint-plugin)

Updates `@typescript-eslint/parser` from 8.71.2 to 8.72.0
- [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases)
- [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/parser/CHANGELOG.md)
- [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.72.0/packages/parser)

---
updated-dependencies:
- dependency-name: "@typescript-eslint/eslint-plugin"
  dependency-version: 8.72.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: npm-minor-patch
- dependency-name: "@typescript-eslint/parser"
  dependency-version: 8.72.0
  dependency-type: direct:development
  update-type: version-update:semver-minor
  dependency-group: npm-minor-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Book lookup returns only audiobook editions, so the /ebook facade scope is always empty

2 participants