Skip to content

fix: handle docker pagination limits - #149

Merged
grantfitzsimmons merged 9 commits into
mainfrom
docker-pagination-limits
Aug 18, 2026
Merged

fix: handle docker pagination limits#149
grantfitzsimmons merged 9 commits into
mainfrom
docker-pagination-limits

Conversation

@grantfitzsimmons

@grantfitzsimmons grantfitzsimmons commented Aug 18, 2026

Copy link
Copy Markdown
Member

Summary by CodeRabbit

  • New Features
    • Docker image tag listings now support improved filtering, ordering, pagination, and configurable page sizes.
    • Supported service images can apply predefined tag filters automatically.
  • Bug Fixes
    • Duplicate tags are removed from results.
    • Docker Hub errors are handled gracefully with empty results instead of failed responses.
    • Tag retrieval now respects Docker Hub’s unauthenticated pagination limits.
  • Improvements
    • State updates now use the same supported tag filters for more consistent results.

@grantfitzsimmons
grantfitzsimmons merged commit 20835b1 into main Aug 18, 2026
2 checks passed
@grantfitzsimmons
grantfitzsimmons deleted the docker-pagination-limits branch August 18, 2026 16:50
@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 16124094-ff8f-43a5-a15c-d9873b799d00

📥 Commits

Reviewing files that changed from the base of the PR and between 252bc54 and e6a7161.

📒 Files selected for processing (3)
  • app/lib/apiUtils.ts
  • app/pages/api/dockerhub/[image].ts
  • app/pages/api/state/index.ts

📝 Walkthrough

Walkthrough

Changes

Docker Hub tag retrieval now supports configurable filters, ordering, bounded pagination, concurrent requests, deduplication, and error handling. New URL and writable-type utilities support the implementation. API and state flows use special filters for specify7-service.

Sequence Diagram(s)

sequenceDiagram
  participant APIHandler
  participant StateSetup
  participant fetchTagsForImage
  participant DockerHubAPI
  APIHandler->>fetchTagsForImage: pass image-specific filters
  StateSetup->>fetchTagsForImage: pass SPECIAL_TAGS for specify7-service
  fetchTagsForImage->>DockerHubAPI: request filtered tag pages
  DockerHubAPI-->>fetchTagsForImage: return tag pages or errors
  fetchTagsForImage-->>APIHandler: return processed tags
  fetchTagsForImage-->>StateSetup: return processed tags
Loading

Changes

Docker Hub tag retrieval

Layer / File(s) Summary
URL and mutability utilities
app/lib/apiUtils.ts
Adds formatUrl for merging non-nullish query parameters. Adds Writable<T> and writable for removing readonly modifiers from types and values.
Filtered and paginated tag fetching
app/pages/api/dockerhub/[image].ts
Adds filter-based requests, bounded pagination, concurrent filter fetching, duplicate removal, and empty results for Docker Hub error responses.
Special filter integration
app/pages/api/dockerhub/[image].ts, app/pages/api/state/index.ts
Passes image-specific special filters to fetchTagsForImage. setState uses the specify7-service configuration.国产自拍
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docker-pagination-limits

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.

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.

2 participants