Repository navigation
feat(server): Harden production document import boundaries - #457
Conversation
Reserve estimated provider costs durably before dispatch, restrict native workers, and bound source-linked page context. Require matched independent quality and operational evidence for production preflight while preserving unknown call receipts and existing contracts. Co-authored-by: Codex <codex@openai.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 731c3fe26e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Keep budget rejections out of completed durable task results so explicit resume can recheck changed limits or a new UTC day without replaying Tencent. Co-authored-by: Codex <codex@openai.com>
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5a6e0f740e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
Keep repeated recoverable cost rejections from consuming correction slots. Retain rejection receipts and prior reservations across explicit recovery. Co-authored-by: Codex <codex@openai.com>
Share a non-secret configuration fingerprint across evaluation and metrics, record it from the deployed service before load submissions, and reject mismatched quality evidence. Keep audit and runtime code hashes distinct. Co-authored-by: Codex <codex@openai.com>
Resolve CVE-2026-118667 reported by the locked-dependency audit. Regenerate the lock with the CI-pinned uv without changing other packages. Co-authored-by: Codex <codex@openai.com>
Summary
Imports now reserve optional estimated provider costs durably before dispatch and apply required OS restrictions to the existing native workers. Missing independent quality or operational evidence produces a blocked preflight instead of a production-readiness claim.
langgraph-sdktransitive dependency to 0.4.6 for CVE-2026-118667, which blocked the locked-dependency audit on Python 3.14 and 3.15.AI assistance: Codex implemented the changes, inspected the complete diff and exercised local checks with substitute providers. Native container checks use real PDFium and LibreOffice. No paid model evaluation or deployment was performed.
Linked issue
Closes #456
Affected areas
AI service, native preprocessing workers, deployment configuration, evaluation/operations tooling and English documentation. No practice App or import Web UI changes; service schema 5, App schema 11 and shared ZIP/API contracts remain unchanged.
Validation
Relevant checks follow CONTRIBUTING.md. Native container, Web and alert rows retain the original
731c3fe2evidence; the review follow-up reruns the complete service checks and connected regression cases.make verify AI_PYTHON=/Users/sheny/.local/share/uv/python/cpython-3.14-macos-aarch64-none/bin/python3.14478d2348: 3,164 tests, 93% total coverage; lock consistency, Ruff, Pyright, 30 evaluation fixtures, recovery probes, source/wheel build and package integrity.make install-locked;make audit AI_PYTHON=/Users/sheny/.local/share/uv/python/cpython-3.14-macos-aarch64-none/bin/python3.14make image-checkmake web-checkmake alerts-checkpython app/scripts/export-contracts.py --check;git diff --check478d2348; the prior-head audit failure was CVE-2026-118667 in SDK 0.4.4. Local audit and complete service verification pass with SDK 0.4.6.Risk and rollout
/usr/lib/libreoffice/program/soffice.binwith its exact version in the supplied image; the shell launcher is incompatible with private IPC. No new dependency, credential destination or client-side model work is introduced.Checklist