feat(techvault): declare Cortex job index as ADR-088 initial service state - #269
Merged
Conversation
Merged
This was referenced Aug 4, 2026
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.
Summary
Converts the TechVault
cortex-index-initone-shot init node into an ADR-088 initial service state declaration. The Cortex job index is now declared as observable state: a materialization-onlycontententry (cortex-job-index-schema) with aservice-search-index-schemaprofile targetingthehive-es, declaring only portablekey/status/relations= exact-token field semantics plus readback scaffolding. The backend's native Elasticsearch materializer establishes and proves the field schema before Cortex starts — no separate init container, and no native index name or vendor ES literals in the SDL. Thecortex_6datastore partition onthehive-es(ADR-048, the observed half) is unchanged; this adds the materialized half (ADR-088). Unblocks Brad-Edwards/aptl#889; thefeattype triggers the release-please version bump aptl pins.This PR also folds in a pre-existing, repo-wide dependency-audit fix (see Changes):
cryptography49.0.0 → 50.0.0 for CVE-2026-69247, which theaudit/verifyCI gates flagged. It is transitive (viaraes/asyncssh/pyjwt) and already red ondev; fixed here rather than deferred.Requirement UIDs
Related Issues
Closes #268
ADR Impact
Changes
cortex-index-initone-shotvmnode, its inlinecortex-index-init-scriptcontent, and itsinfrastructuretopology row (nothing else referenced it).content.cortex-job-index-schema(dataset, targetthehive-es) with theservice-search-index-schemaprofile,target_service_ref: nodes.thehive-es.services.elasticsearch, and exact-token field semantics forkey/status/relations.propositions,assertions,observation_boundaries,evidence_requirements— mirroring the RAESinitial-service-stateexample; the evidence requirement carries the RAES-3.3.0-requiredintegrity/retention/loss_disclosurefields the issue snippet omitted.associated-artifacts.jsonviatools/refresh_pack_sdl_binding.py.tests/test_techvault_pack.pytopology guards (node/infrastructure counts 38→37, content inline/sourced/materialized partition) and addtest_cortex_job_index_schema_is_adr088_initial_service_stateas a structural invariant guard.auditgate): bumpcryptography49.0.0 → 50.0.0 inrequirements/runtime.txtfor CVE-2026-69247, with authoritative PyPI hashes; verified by a--require-hashesinstall +pip-audit(clean).cffi 2.1.0already satisfies 50.0.0'scffi>=2.0.0; the frozenrecovery-v2.0.2.txtsnapshot is left untouched.Test Plan
Full completion command green (812 tests + raes-pack-validate/release check across repo and packs) and policy command green. Pre-push codex and test-quality reviews both clean (0 findings).
raes-pack-validate --pack packs/techvaultconfirms the SDL parses under the pinned raes 3.3.0 with the new materialization and readback sections.pip-auditagainst the updated closure reports no known vulnerabilities.Ground Control Checks
gc_evaluate_quality_gatespasses or is unchanged by this repo-only changegc_run_sweepreviewed; findings fixed or recorded with rationaleTraceability
Checklist
docs/CODING_STANDARDS.md)@Auditedon new entities if applicableDocumentation
Verified unchanged: no documentation surface in scope.