Repository navigation
[VERIFY-01] Remediate all exact-main audit findings #96
Copy link
Copy link
Closed
Description
TASKS IDs: UI-GATE, LIVE-01, SEC-REVIEW, STABILITY, DEMO-REHEARSAL evidence
PRD: 0.3-0.4, 6.1, 11-17
Related issues: #45, #65, #89
This issue owns remediation of every concrete defect reproduced during the 2026-08-31 exact-main audit.
Product and security scope:
- bound durable event, run, message, and Shepherd collections so reaching the schema limit cannot make state permanently unwritable
- reject hostile Origin and Host mutation requests in tokenless loopback mode without breaking the local demo
- resolve the non-loopback tokenless Compose posture carried by [SEC-REVIEW-2] Independent trust-boundary review and the carried compose exposure #89
- keep Volcengine credentials out of Terraform state and cloud-init user data
- remove the Project Group mention caret race
- give every Settings numeric input an unambiguous accessible name
- remove the 1280x800 Create Agent fractional clipping
Verification and harness scope:
- make all 38 committed Playwright tests green at 1280x800 and 1440x900 without weakening assertions
- make the legacy live starter gate honor the Shepherd-route default and resolve codex through PATH safely
- confine the live model-review gate to one current-tree test and one external request
- make the sparse live-runtime gate volume-aware on the affected macOS container host while preserving sandbox, cleanup, and credential boundaries
- add an enforced repository coverage gate with an 80 percent minimum
- run pinned Terraform fmt and validate for deploy/volcengine
Documentation scope:
- reconcile TASKS, SHEPHERD, LOCAL_POC, FIXES, BUILD_LOG, and SHEPHERD_TEST_REPORT with exact observed evidence
- record [UI-GATE] Complete the final six-surface accessibility and visual gate #45, [LIVE-01] Run the exact-head sparse live Shepherd Mission and legacy Agent continuity gate #65, and reopened [SEC-REVIEW-2] Independent trust-boundary review and the carried compose exposure #89 ownership and remaining independent gates truthfully
- do not self-claim independent UI or security review
Primary files: database schema and bounded append paths; server auth/config and tests; Compose and Volcengine deployment; Project Group, Settings, Create Agent UI and Playwright tests; live runtime/model-review harness; package and Vitest coverage configuration; project evidence documentation.
Acceptance:
- every new behavior is introduced by a causal failing test and verified green
- complete Playwright matrix passes at both required viewports
- live gates are bounded, current-tree-only, redacted, and pass on this host when code can make them portable
- repository coverage command enforces at least 80 percent
- Terraform format and validation pass with a pinned version
- npm audit and npm run check pass
- documentation contains no known stale status for this audit scope
- branch starts at current origin/main and is delivered through a linked PR.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels
Activity
[-][VERIFY-01] Close exact-main browser, runtime, coverage, and infrastructure gates[/-][+][VERIFY-01] Remediate all exact-main audit findings[/+]sanjey99 commentedon Aug 31, 2026
Taking over as authorized for consolidated remediation on branch fix/96-release-readiness-audit, based on current origin/main f935fdb. Primary ownership is limited to the concrete exact-main audit findings and their causal tests/evidence. No UI redesign. Issue #96 is the umbrella tracker and this issue remains linked for its own acceptance semantics. Independent review will not be self-claimed.
sanjey99 commentedon Aug 31, 2026
Full candidate evidence is now pushed in PR #97 at bb1a1df. All concrete issues identified during the exact-main audit are remediated: browser functional races/accessibility/overflow, durable capacity and atomicity, tokenless/public-bind security, exact-tree volume-aware live construction, executable 80 percent coverage, reproducible Terraform validation, Terraform credential-state removal, and stale release documentation. Observed gates: npm run check root 26/26, Server 856 passed plus 3 opt-in skips, Web 20/20, strict types/builds; clean Server coverage 86.15/80.05/91.46/88.03 percent and browser 88.50/80.18/87.64/92.37 percent; Playwright 48/48 across both required viewports; Terraform valid; zero-spend live preflight pass; npm audit zero. Hosted PR checks are running. External Ark inference and independent U/security review remain explicitly separate evidence gates.
sanjey99 commentedon Aug 31, 2026
Final pushed head 96eecb9 is now hosted-green. Required checks run 33390293341 passed in 2m34s and Node 22 / npm run check run 33390293339 passed in 3m16s. The branch and PR are clean/mergeable pending final status verification. The only test request blocked by policy is the external Ark gate: explicit approval must acknowledge that repository-derived prompts and scoped source code will be sent to the configured Ark endpoint and live model capacity will be consumed.
docs: record issue 96 protected-main evidence