Skip to content

feature: modify document page and add Codex Auto Review Action - #1

Closed
Dynamite2003 wants to merge 11 commits into
devfrom
frontend/document
Closed

Dynamite2003 wants to merge 11 commits into
devfrom
frontend/document

Conversation

@Dynamite2003

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings November 20, 2025 08:07

Copilot AI 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.

Pull Request Overview

This PR implements client-side pagination with deduplication for the library page and adds a development helper script. The PR title mentions adding a "Codex Auto Review Action," but the workflow file is not included in the actual changes.

Key changes:

  • Introduces client-side pagination and deduplication logic for the uploaded papers library
  • Adds a bash script (scripts/dev.sh) to streamline local development environment setup
  • Includes theme management for the auth page
  • Adds generated OpenAPI specification file for API documentation

Reviewed Changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
scripts/dev.sh New development helper script that orchestrates Docker services, backend, Celery worker, and frontend startup
frontend/src/app/library/page.tsx Refactored to fetch all papers upfront, deduplicate them client-side, and implement local pagination instead of server-side pagination
frontend/src/app/auth/page.tsx Added useEffect to manage theme state (force light mode on auth page with proper cleanup)
backend/openapi.json Generated OpenAPI 3.1.0 specification documenting all API endpoints, schemas, and security schemes
README.md Updated contributing section to reference the (non-existent) Codex Auto Review workflow and fixed markdown separator

Comment thread frontend/src/app/library/page.tsx
Comment thread frontend/src/app/library/page.tsx
Comment thread frontend/src/app/auth/page.tsx
Comment thread README.md
Comment thread frontend/src/app/library/page.tsx
@Dynamite2003
Dynamite2003 deleted the frontend/document branch November 27, 2025 01:54
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