flex-auth/workplans/FLEX-WP-0019-layer-model-conformance.md
tegwick 7d7b4531d1
All checks were successful
CI Smoke / host-smoke (push) Successful in 0s
CI Smoke / container-smoke (push) Successful in 1s
Point FLEX-WP-0019 review at the implementation commit it assessed
C-21 flagged the ready workplan as possibly stale: reviewed against 5753b47,
HEAD had moved. Checked rather than assumed — nothing since 5753b47 touches Go
or schemas. Every commit between is documentation or work-record only, and the
last implementation change is d402408 (2026-08-23).

reviewed_against_commit now names the implementation state reviewed rather than
the HEAD at review time, so it moves when the code moves, which is when the
review needs redoing. C-21 compares against HEAD and cannot distinguish a doc
commit from a code one, so it will keep warning softly on this plan; the record
is accurate and that is the trade taken deliberately.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_012sgN4GH5ZYT8pJVkCR6dcP

Assistant: claude-code
Assistant-Model: opus
Assistant-Process: 4014348@bnt-lap001
Assistant-Session: a993abda-65a0-4ea8-8ccd-0fcd78c92ac0
2026-08-29 15:00:29 +02:00

170 lines
7 KiB
Markdown

---
id: FLEX-WP-0019
type: workplan
title: "Layer model v0.7 conformance: provenance, lifetimes, deadlines, and the decision contract"
domain: infotech
repo: flex-auth
status: ready
owner: codex
topic_slug: netkingdom
planning_priority: P1
planning_order: 119
created: "2026-08-29"
updated: "2026-08-29"
reviewed_at: "2026-08-29"
reviewed_against_commit: "d402408"
reviewed_note: >-
d402408 is the last commit to touch implementation (Go sources and schemas);
the assessment was performed with the working tree at 5753b47, and every
commit between the two is documentation or work-record only. This field names
the implementation state reviewed rather than the HEAD at review time, so it
moves when the code moves — which is when the review actually needs redoing.
origin: history/2026-08-29-layer-model-v0.7-alignment-review.md
quality_dor: DoR-Ok
quality_dor_at: "2026-08-29"
quality_dor_by: "claude-code"
quality_dor_note: >-
Assessed against policies/work-item-dor.md. Shared criteria: outcome per task,
scope-out stated (G7 needs nothing, G8 stays with FLEX-WP-0017), owning repo
and executor in frontmatter, the one internal dependency stated (T02 before
T04) with no external hard blocker, and a verification signal per task.
Workplan criteria: goal stated, six tasks decomposed, and repo-state review
recorded above — each obligation was checked against code at 5753b47 in the
alignment review, not against documentation.
state_hub_workstream_id: "84b9dc5a-71f2-5c70-ace6-78242b13d0f1"
---
# FLEX-WP-0019 - Layer model v0.7 conformance
The NetKingdom Security Layer Model was accepted at v0.7 on 2026-08-29 and
`INTENT.md`/`SCOPE.md` were aligned to it. The alignment review
(`history/2026-08-29-layer-model-v0.7-alignment-review.md`) identified six gaps
between what flex-auth now states and what it implements. This workplan closes
them.
Two are conformance obligations rather than features: the registry-snapshot
digest (§9.7.2 promotes it from housekeeping to a prerequisite, on flex-auth's
own argument) and the per-input-class visibility deadlines that depend on it.
Ordering is load-bearing. **T02 precedes T04** — publishing a visibility
deadline for registry-borne facts before provenance can identify the snapshot
would publish a number nobody can check.
## Assert the layer declaration mechanically
```task
id: FLEX-WP-0019-T01
status: todo
priority: medium
state_hub_task_id: "f72b1305-114e-5ba2-b84c-f6cc0b524178"
```
`INTENT.md` frontmatter carries `layer: Engine`, `role: PDP` per §11. Add a
conformance check and test asserting the declaration parses, states a layer in
the §3 vocabulary, states a role because the layer is Engine, and that no
Tooling client exists in the tree — the Engine-side analogue of ops-warden's
`scripts/check_layer_conformance.py`, which is offered estate-wide as a
reference form.
Closes G1's remaining half. Wire it into `make test`.
## Add the registry-snapshot digest to decision provenance
```task
id: FLEX-WP-0019-T02
status: todo
priority: high
state_hub_task_id: "7a980074-8488-5ab1-9202-60878adb261d"
```
`DecisionProvenance` carries no digest of the registry snapshot that supplied
resource, subject, and relationship facts, so a decision that turned on registry
content cannot be replayed from its own record (§6.1). This is flex-auth's one
declared gap in §13 and a conformance prerequisite under §9.7.2.
Reuse the canonical-JSON plus SHA-256 pattern already in `pkg/api/canonical.go`
over `registry.Snapshot`, which is already deterministic and has a test asserting
it (`TestStoreLoadsAndSavesDeterministicSnapshot`). Thread the digest into
`DecisionProvenance`, through the standalone engine and through every delegated
adapter that shares the canonical constructor, and into
`schemas/decision_envelope.schema.json`. Add regression coverage that a changed
snapshot changes the digest and that two decisions over the same snapshot agree.
## Give every allow an explicit lifetime
```task
id: FLEX-WP-0019-T03
status: todo
priority: high
state_hub_task_id: "9d9c0e7a-56e2-5c71-9110-cea973243c22"
```
§9.7.1 requires every allow to carry an explicit lifetime — a TTL, or a binding
to a session or obligation that ends. `DecisionEnvelope` has no lifetime field;
`ActionAuthorizationValidity` has one but belongs to the durable approval object,
not the decision.
The struct field is trivial; the semantics are not. flex-auth has no session
concept, so the first honest shape is a **policy-package-declared TTL with an
explicit default**, and a deny for any allow that would otherwise have no stated
end. That is a policy-surface change: it needs package schema support, fixtures
covering declared TTL, defaulted TTL, and the no-stated-end deny, and it must
appear in the decision record and the published schema.
## State revocation visibility deadlines per input class
```task
id: FLEX-WP-0019-T04
status: todo
priority: high
state_hub_task_id: "d5917b24-2efb-503e-9a9d-837702cafc1c"
```
§9.7.2 requires a PDP to state a deadline **per input class**, because a decision
is a join over sources with unrelated refresh behaviour. Enumerate the classes
flex-auth actually joins — approval-claim freshness, registry snapshot cadence,
policy package activation, directory ETag — and state a deadline for each, with
the mechanism that bounds it rather than an aspiration.
Depends on **T02**: the registry deadline is unfalsifiable until provenance can
identify the snapshot a decision read. Publish as
`docs/decision-input-freshness.md` and reference it from `INTENT.md`.
## Publish the decision-record schema as flex-auth's contract
```task
id: FLEX-WP-0019-T05
status: todo
priority: medium
state_hub_task_id: "f7f501d7-6862-5542-88f7-78b704524706"
```
§17 moves the decision-record schema from Taxonomy to `access-engine`, on
flex-auth's own argument that a decision record is the PDP's output artifact.
`schemas/decision_envelope.schema.json` exists and describes `binding` and
`provenance`, but carries no lifetime (T03), no snapshot digest (T02), no
policy-package **digest** — name and version only — and no input-claim digests.
Complete it, version it as a published contract rather than an internal shape,
and document it in `docs/` as the artifact consumers may rely on. Taxonomy holds
only the shared field vocabulary the claim schema references; this schema is
flex-auth's.
## Publish the request digest as the replay test
```task
id: FLEX-WP-0019-T06
status: todo
priority: medium
state_hub_task_id: "bc109ee3-14b0-5603-a655-0d7376c2a41c"
```
§6.4.2 makes replay permitted **iff the canonical request digest matches and the
decision's lifetime holds**, and names flex-auth's digest as the mechanical test.
The digest is computed and regression-tested but is documented nowhere a consumer
would look.
Publish what it is computed over, what normalization applies, its stability
guarantees across versions, and worked examples of a permitted retry and a
refused replay. Consumers cannot honour §6.4.2 against an undocumented digest,
and negative caching under §6.4.2 needs the same guidance.