Enforce StarIntel v0.9 documents at ingest #116
No reviewers
Labels
No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
nsaspy/starintel-server!116
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "agent/v09-document-runtime"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
What
Make StarIntel 0.9.0 the canonical document runtime across server build and ingest paths.
star-clruntime;schema_version, not the integer documentversioncounter;star-clv0.9 JSON-schema validator throughstar.documents:validate-v09-document;Why
The repository schema lock already declares v0.9.0, but QLot was pinned to a 0.7.3
star-clrevision and Nix was pinned to 0.8.0. The HTTP boundary also compared the documentversionfield against the schema version, which is incorrect in v0.9 (schema_versionis the spec revision;versionis an integer document revision).Related
Advances #59: canonical document ingest is schema-aware and validates v0.9 before publish/persist. This does not implement the full versioned batch/target lifecycle API from #59.
RAGE/TDD evidence pin for exact head
45c493f26d8e763dc462e122be5c9cccf46f1d7d(Smoke run32713779855, synthesized merge08a1bbccce79bc600a8b21cea7886826c5c86917).The hermetic
unit-testsjob is red, but the immutable uploaded artifact9515137597does not contain a failing test. Every required suite reportsfailed=0 skipped=0, including HTTP boundary (22/22), authorization policy (18/18), gserver client (19/19), and runtime lifecycle (4/4); the log ends with only 7 SBCL STYLE-WARNING conditions and noTest system ... failed/ unhandled error. The workflow step deliberately capturesnix run .#star-unit-tests's exit status and exits with that status, so the evidence currently proves a runner/process exit-status discrepancy, not a failing hermetic assertion.Do not weaken or skip tests to green this. Reproduce the exact
nix run .#star-unit-testsexit status at this head and fix the runner/process cause if deterministic. The separate service-backed integration job is also red and still requires its own deterministic failure evidence before any product-code repair is justified.RAGE diagnostic follow-up: the service-backed integration failure is reproducible on the prior exact head, but the old Smoke workflow discarded its integration log. Commit
c65871fbbf55a3f5db7dd1d2fb6d6a5d35209003changes CI only: it capturesnix run .#star-integration-tests, preserves the exact exit status, tails the log, and uploads the complete service-backed integration log as a 1-day artifact. No product assertion or validation is weakened.Important remote-state change: this PR is now materially stale/diverged from current
master; GitHub currently reports it non-mergeable. Current master also contains later Smoke/Clouseau integration-harness work. Do not diagnose or patch v0.9 product behavior from the old red badge alone. First use the new immutable integration artifact if CI runs this head; then reconcile the branch with current master and require fresh exact-head gates before merge.RAGE reconciliation: closing this PR as superseded by #119. PR #119 re-cut the v0.9 canonical-runtime work against current
master, reached final head922bda3c8bb346b4eb1700281417d7a018253621, passed Smoke #404, Container Stack #405, Canonical StarIntel schema lock #290, and Operational salvage #235 on that exact head, and merged tomasteras52948721d8981e9aa5ccad4efd77185e52af845bon 2026-08-25. Do not continue product fixes or CI diagnosis on this stale branch; follow-up work should start from currentmasterand preserve #119's canonical v0.9 boundary.Pull request closed