Repository navigation
Conversation
knqiufan
requested review from
AlexStocks and
hidb4ai
and removed request for
zhanghuidinah
October 13, 2026 09:26
AlexStocks
approved these changes
Oct 15, 2026
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.
Which issue or RFC does this PR close?
Refs #1932 and draft implementation #1956. This is a design proposal; both remain open.
Rationale for this change
Image capture needs durable original bytes, asynchronous interpretation and traceable evidence for Memory. The proposal fits those responsibilities into the existing Source, Artifact and Supervisor contracts. Shared media storage lets later audio/video handlers reuse asset identity, request recovery and capacity accounting without creating a separate table set for every modality.
Delayed interpretation also publishes a new Source entry for older evidence. Reconciliation needs comparable, statement-specific ordering support for both incoming evidence and recalled Memory so completion time cannot turn an old fact into a new one.
What changes are included in this PR?
7f38426eand a responsibility comparison against the experimental implementation.image-analysisArtifact family and deterministic interpretation-publication Sources for Source-driven Atomic Memory. Existing revisions, lineage, authorization, inspection and Supervisor scheduling are reused.pc_media_assets,pc_media_processing_requestsandpc_media_capacity. The RFC defines common identity/lifecycle/accounting fields, bounded versioned handler data, typed readback and the boundaries for reusing existing storage.processing_kindin request identity and active-request uniqueness. Model/profile changes cannot bypass same-purpose exclusion; handlers keep their own validation, Source Definitions, Artifact Families and execution budgets.Are there any user-facing changes?
Documentation only. This PR proposes future behavior and makes no runtime, API, database schema or dependency changes. Shared media storage and Memory ordering extensions are implementation requirements, not existing runtime capabilities. Audio/video ingestion is outside the initial release. #1956 remains a draft prototype pending RFC acceptance and implementation alignment.
How was this change tested?
uv sync --lockedandpnpm --dir website install --frozen-lockfile.pnpm --dir website lint.CI=true pnpm --dir website build: 13 website tests passed; link checks, content generation and production build passed; static export verification confirmed 920 public pages and their internal links.SKIP=ty-check uv run prek run -a: all other hooks passed. The native Windows type-check hook was excluded because unchanged code has existing POSIX-only API diagnostics;uv run --no-sync ty check --python-platform linuxpassed separately. No CI workflow was changed.git diff --check; strict UTF-8 without BOM; 14 relative links and 22 pinned source targets verified; bilingual code identifiers and design rules checked for consistency.These checks validate the documents. Media behavior, the capacity alternatives and actual host/model support require implementation and subsequent qualification.
AI usage statement
OpenAI Codex assisted with source inspection, design comparison, bilingual drafting and documentation checks. The author is responsible for the proposal.