Skip to content

旧舟归港,云门初启 · Integrate server preview#35

Merged
eric8810 merged 2 commits into
mainfrom
codex/n2-server-workspace
Jul 22, 2026
Merged

旧舟归港,云门初启 · Integrate server preview#35
eric8810 merged 2 commits into
mainfrom
codex/n2-server-workspace

Conversation

@eric8810

@eric8810 eric8810 commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

  • migrate the REST API from PR feat(server): add REST API + Docker for light-ocr #24 and chatre7/light-ocr-server into the N2 workspace
  • keep the server private at 0.1.0 with a versioned /api/v1 contract
  • retain bounded uploads, OcrError HTTP mapping, graceful shutdown, and contributor attribution
  • build the preview image from the root workspace lock on Linux x64 or arm64
  • split CI by changed paths so workspace changes use a focused Node job and only native-impacting changes run the full Core build

Verification

  • npm ci plus npm test and npm run typecheck
  • docker compose config
  • npm pack --dry-run --workspace @arcships/light-ocr-server
  • production workspace install with dev dependencies omitted

The Dockerfile could not be built locally because the Docker daemon is not running. Compose parsing and the exact production dependency layout were verified without it.

Provenance

Follows up #24 at the N2 migration point requested in that PR. The original contributor is retained in the commit trailer and package README.

eric8810 and others added 2 commits July 22, 2026 23:47
Migrate the REST and Docker implementation promised by PR #24 into the N2 workspace. Keep the package private at 0.1.0 while its versioned HTTP contract is evaluated.

Bound uploads before buffering, preserve OcrError semantics, and reuse the representative native build for one integration request instead of duplicating the CI matrix.

Co-authored-by: chatre7 <chatree.yosnak@gmail.com>
Run the full Core job only when native, model, corpus, or release tooling paths change.

Give workspace and server changes a focused Node job that still performs a real OCR request with exact published native dependencies.
@eric8810
eric8810 merged commit eddee23 into main Jul 22, 2026
1 check passed
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