conventions-ingestion
GitHub定义外部数据摄入的安全规范,确保幂等、回放安全与持久化进度。涵盖身份稳定、检查点管理、原子事务及陈旧工作保护,适用于导入、同步、Webhook等异步处理场景。
Trigger Scenarios
Install
npx skills add stella/stella --skill conventions-ingestion -g -y
SKILL.md
Frontmatter
{
"name": "conventions-ingestion",
"description": "Apply when building or reviewing external ingestion, imports, connector polling, webhooks, extraction workers, sync cursors, checkpoints, or repair jobs. Enforces replay safety, idempotency, durable progress, and bounded recovery."
}
Replay-Safe Ingestion Conventions
Apply to any workflow that turns external or asynchronous input into durable stella state: paginated imports, connector sync, webhooks, file extraction, queue workers, migrations, and repair scans.
Target Property
A retry, duplicate delivery, worker restart, or overlapping run must converge to the same durable state as one successful run. Idempotency is one ingredient; replay safety also requires correct checkpoint ordering, durable retries, and protection from stale work.
Required Design
- Stable identity. Give every source item a stable, tenant-scoped identity. Enforce it with a database unique constraint whose leading columns preserve the tenant or source boundary. Do not rely on a hash collision check alone.
- Idempotent persistence. Upsert, claim, or transition by stable identity. Reapplying the same input must not duplicate rows, counters, notifications, or other effects.
- Explicit outcomes. Distinguish terminal outcomes (applied, unchanged, deliberately rejected) from retryable failures. A skipped item is terminal only when losing it is intentional and auditable.
- Checkpoint last. Advance a cursor, watermark, or checkpoint only after every earlier item is terminal or has a durable retry record. On an ambiguous failure, hold the old checkpoint and replay.
- Atomic database batches. For database-only work, persist the items and
checkpoint in one short transaction with
commitReplaySafeIngestionBatchfromapps/api/src/lib/replay-safe-ingestion.ts. - External side effects. Object storage, search indexes, email, and remote APIs cannot join the database transaction. Make the database record the source identity/fingerprint and retry state first; use deterministic object keys or provider idempotency keys. Persist the cursor only after all page work is durable.
- Compare-and-set cursors. Capture the cursor loaded at run start and
require it in the checkpoint update. A stale run must return the persisted
winner, never overwrite newer progress. Public corpus ingestion uses
advanceCorpusIngestionCheckpointfromapps/api/src/lib/corpus-ingestion-checkpoint.ts. - Stale-work protection. Mutable inputs need a source version or content fingerprint. A late worker must compare the claimed version before overwriting newer state. AI outputs also include schema, model, prompt, and parser versions in their identity/provenance.
- Durable execution. Do not rely on detached promises or process memory for required work. Use a durable queue/outbox and deterministic job identity. Add a bounded repair scan when enqueue and commit cannot be atomic.
- Bounded recovery. Repair scans and list reads use cursor pagination and configured limits. Workers remain stateless and safe under concurrency.
- Owned schema. The vertical slice owns its source, item, attempt, and failure tables. Shared code provides transaction and identity primitives, not a cross-domain ingestion framework.
- Bounded external calls. Every remote request has an explicit timeout, bounded retry policy with jitter, provider-aware rate limiting, and a maximum concurrency. Persist retry state; do not hold a database transaction while waiting on the provider.
- Poison-item isolation. One malformed or permanently rejected item must not stall an entire source forever. Persist the item identity, classified terminal/retryable outcome, sanitized error context, and operator-visible repair path before allowing later progress.
- Accounted-for source fields. Every field a source states on a page the adapter already fetches is stored, or excluded with the reason. See below.
- A publisher is not a court. The deciding court comes from the record:
the court code in its ECLI first, then the record's own court field,
resolved through the shared resolver
(
apps/api/src/lib/case-law/cz-ecli-courts.tsfor Czech sources). Never a per-adapter constant, even where the portal is believed to serve one court's decisions: portals republish other courts, and the court name is what authority weighting is read off. An inventory declaration must name the field the value is read from, and a value produced from a literal satisfies no declaration.no-literal-decision-courtrejects a literalcourtin a case-law adapter.
Checkpoint Boundary
Direct Drizzle writes to syncCursor are banned by
no-direct-ingestion-checkpoint-write. Public corpus cursors go through
advanceCorpusIngestionCheckpoint; database-only batches keep the write inside
the persistCheckpoint callback passed to commitReplaySafeIngestionBatch.
The lint rule enforces the visible boundary; it does not prove that preceding
external effects are durable.
Source-Surface Census
A page an adapter never fetched is invisible to any inventory of the pages it
did. So the declaration comes in two steps, and the first is the surfaces:
SourceAdapter.sourceSurfaces is required, and it is total over the addresses
the publisher serves for one decision — SOURCE_SURFACES as a kebab-case list,
then the map as const satisfies Record<<that union>, SourceSurfaceDisposition>
as the surfaces of a literal written as const satisfies SourceSurfaceCensus.
Each surface is one of three things:
storedSourceSurface(part)— fetched with the decision and kept as that envelope part. The conformance suite reads the part back out of an envelope this repository can produce, so an adapter with no such capture cannot declare one;excludedSourceSurface(reason)— the same constructor discipline as the field inventory: a blank reason does not compile, and "not fetched today" is not a reason. A rendering of a payload already kept, a corpus-wide index, a query-scoped export and a page the publisher's robots policy disallows are;backlogSurface(adapter, reason)— it belongs in the row and is not there yet. Only an adapter already onsource-surface-backlog-baseline.jsoncan be named, each entry is listed there, andsource-surface-census.test.tsfails both ways, so the set only shrinks. Recording a surface deletes its line.
Source-Field Inventory
A field an adapter never noticed is indistinguishable from one it decided to
leave. Case-law adapters therefore declare what their source states, and the
declaration is part of the contract: SourceAdapter.sourceFields is required,
so an adapter without one does not compile.
An inventory has three parts, in the adapter beside the readers it mirrors:
SOURCE_FIELDS, the list of names the source labels on the per-decision pages this adapter parses,as const;- a disposition map written
as const satisfies Record<<that union>, SourceFieldDisposition>, so the map is total by type and a new name without a decision does not compile. Each entry is{ disposition: "stored", target }(a metadata key, a result field, the parsed document, or the row's identity), orexcludedSourceField(reason), where the reason says what the field is and why the row does not carry it. "Not read today" is not a reason; duplicate of a stored field, derived elsewhere, no field on the row, and data minimization are. The constructor is the only way to write an exclusion, and a blank reason does not compile; listSourceFields(parts), which reads the stored envelope back and answers what the publisher labelled across it. The whole envelope, not one page: a source states fields on the listing row as well as on the detail payload, and a reader given one of them declares the others out of scope by accident.
source-field-inventory.test.ts drives every registered adapter from the
registry: each adapter's fixture is built, its stored envelope goes
through its own listSourceFields, every name that comes out must be in the
map, and every field the map stores must be on the decision built from that
fixture, at the target the disposition names. A field on the page that is in
neither set fails with its name, and so does a field the map declares that the
envelope never states. Its coverage map is total over the registry, so a source
registered without a fixture does not compile.
Refresh a fixture from the live page when the source changes. The suite certifies the adapter against the page it is given, so a fixture that stopped matching the publisher certifies nothing.
Raw Holds Every Fetched Response
An inventory decides what is read; the stored raw decides what can still be
read later. Where a source serves one decision across several responses (a
detail page beside the document), store all of them, with
encodeSourceRawEnvelope and SOURCE_RAW_ENVELOPE_CONTENT_TYPE, naming each
part by its role. A raw that holds only the page the parser read makes a field
captured later unrecoverable for every stored row: replay can only re-read what
was kept.
The envelope holds text. A response the adapter keeps as bytes goes through
sourceRawBytes, which the pipeline stores instead of sourceRaw, so an
adapter that sets both loses the payload that names the decision.
reparseStoredRaw decodes with decodeSourceRawEnvelope and handles null,
which is what a row stored before its adapter had an envelope reads as. The
shapes those rows hold are registered per adapter in LEGACY_RAW_SHAPES, so a
reader knows which payload it is looking at and a migrated adapter deletes its
line. Bump the adapter's entry in PARSER_VERSIONS when the replay's output
changes, and state in the pull request what a replay does and does not
backfill: rows stored before the change hold what they held, and only a
re-crawl adds to them.
Verification
Test the behavior that types and lint cannot prove:
- replay the same batch and assert the same fixed point;
- fail item persistence and assert the checkpoint does not advance;
- fail checkpoint persistence and assert database item writes roll back;
- deliver duplicates concurrently and assert one durable effect;
- crash after a remote side effect and before acknowledgment, then replay;
- finish stale work after a newer version and assert it cannot overwrite;
- leave an enqueue gap and assert the bounded repair scan finds it;
- exhaust the retry budget for one poison item and assert later items still reach a durable terminal state without silently dropping the failure;
- overlap two workers at the provider concurrency limit and assert calls stay bounded and the persisted winner cannot be overwritten;
- read a source fixture back through the adapter's own
listSourceFieldsand assert every field it states is stored or excluded with a reason.
Prefer invariant and state-machine tests over one example retry.
Existing References
- Upload finalization:
apps/api/src/handlers/uploads/update.ts - Case-law ingestion:
apps/api/src/handlers/case-law/ingestion/pipeline.ts - Legislation ingestion:
apps/api/src/handlers/legislation/ingestion.ts - Hosted usage webhook deduplication:
apps/api/src/lib/hosted-usage-provider/webhook-store.ts
These are examples, not blanket proof: audit each new side effect and checkpoint independently.
Version History
- e41bf51 Current 2026-09-22 09:03
- 7b52e43 2026-09-09 03:28


