Skip to content

Commit 834af09

Browse files
committed
FEATURE-ci-v3-integration: Mark local task complete
1 parent c794aaa commit 834af09

2 files changed

Lines changed: 29 additions & 2 deletions

File tree

‎specs/tasks/FEATURE-ci-v3-integration.md‎

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,8 @@
11
# FEATURE-ci-v3-integration
22

3-
Status: locally implemented and tested; local verification recorded in the task evidence;
4-
actual GitHub PR verification and final validation remain root-owned.
3+
Status: Complete (local implementation, validation and finalization).
4+
Current-source GitHub platform and full release receipts remain CI / v3 PR owned;
5+
completion does not claim release readiness.
56

67
Scope: native CI/replay, installed platform prerequisites, bounded fuzz campaigns,
78
selected-archive release acceptance and publication enforcement. No production
@@ -41,3 +42,12 @@ focused public TLS run using the existing isolated network path passed all four
4142
shared/static and managed/external cells (82 checks each). This proves the
4243
affected prerequisite correction; it does not turn the failed whole matrix into
4344
a passing receipt. The fresh complete 18-gate release run remains pending CI.
45+
46+
Final local validation (2026-10-23): six Groundwork iterations closed 18 blocking
47+
findings. The final real symlink-root and real ps regression passed 22 focused
48+
and 55 affected-module tests. Earlier source-bound native TLS, sanitizer replay,
49+
workflow and release-contract evidence is retained in the durable validation
50+
record. Current source c794aaac is pushed to PR #381; Verify Build and ChangeLog
51+
pass, while native/fuzz, CodeQL and platform/archive workflows remain pending.
52+
Prior failed receipts remain unsuccessful. Two approved unworked observations
53+
are persisted under specs/unworked_review_issues/. No tags or releases created.
Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
# Unworked Review Issues
2+
3+
**Run:** 2026-10-23 03:13:10
4+
**Task:** FEATURE-ci-v3-integration
5+
**Total:** 2 (0 critical, 1 major, 1 minor)
6+
7+
## Major
8+
9+
1. [ ] **cloud-infrastructure-reviewer** | `.github/workflows/v3-native.yml:77` | iac-anti-pattern
10+
[Reliability — Failure propagation] New tee pipelines at lines 77 and 86 mask bootstrap/Configure failures because the step has neither shell: bash nor set -o pipefail. GitHub implicit Linux shell is bash -e. A bounded execution of this exact script with bootstrap exit17 and Configure exit23 returned 0 and continued after bootstrap failed. New tests at scripts/test-v3-ci-workflows.py:92 and genuine evidence manually supply pipefail, masking the mismatch.
11+
*Recommendation:* Declare shell: bash on the step or enable pipefail in its script. Derive behavioral test shell flags from the workflow; verify bootstrap failure stops before Configure and Configure retains its nonzero status with logs uploaded. See https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-syntax#jobsjob_idstepsshell
12+
13+
## Minor
14+
15+
2. [ ] **code-simplifier** | `.github/workflows/v3-release-acceptance.yml:486` | code-structure
16+
The publisher constructs its complete expected authorization binding in four densely packed lines, including an embedded producer-artifact comprehension. This makes individual environment-to-binding assignments harder to audit alongside the subsequent identity checks.
17+
*Recommendation:* Optionally format this dictionary with one binding field per line and construct the producer_artifacts mapping in a clearly named local variable before assembling expected. Preserve every field, comparison and environment lookup, and keep the publisher code inline so its no-downloaded-code boundary remains unchanged.

0 commit comments

Comments
 (0)