Conversation
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What changed
The v2.1.0 candidate lacked real Korean and combined-language runtime smoke coverage, and its architecture documentation still described OCR as internal after the Image → Text UI shipped. This PR adds deterministic English/Korean fixture coverage for
korandeng+kor, locks OCR runtime loading to the Image → Text route, corrects the release documentation, and records the complete Sprint 16C release-hardening evidence and unpublished v2.1.0 release-note draft.No product behavior, dependency, CSP, lockfile, or prepared OCR asset changed.
Release result
GO. No BLOCKER or HIGH issue remains. The candidate is ready for a separate
v2.1→mainrelease-promotion task after this PR is reviewed and merged.The documented remaining limitation is device-dependent large-image latency: a 3000 × 2000 fixture completed correctly in the constrained Chromium QA profile but took about 14 minutes. Cancellation remains available, and no arbitrary input limit was added.
Validation
npm ci --ignore-scripts— 0 vulnerabilitiesnpm run prepare:ocr— regenerated assets produced no tracked asset diffnpm run buildnpm testnode tests/ocr-smoke.test.mjs— English, Korean, and combined OCR passedgit diff --checkTracks #79.
This PR targets
v2.1. It does not promote tomain, publish a tag or release, or enable auto-merge.