Repository navigation
[CI] add missing tsconfig.ci.json for TypeScript type-check job - #4
vortsghost2025 wants to merge 3 commits into
Conversation
…solated tests - Replace scaffold with strict AUDIT_ONLY/PREPARE/IMPLEMENT/PUBLISH/DEPLOY modes - Unknown modes fail closed - AUDIT_ONLY reads bounded metadata only, never mutates repository, index, ledger, or services - PREPARE creates clean worktree and contract without commit/push/PR - IMPLEMENT validates allowlist, forbidden paths, runtime state, tests, and diff - PUBLISH stages named files only, blocks default branches, scans for secrets - DEPLOY remains disabled by default - Injectable command runner for isolated testing - 14 passing isolated tests using temp repos, temp state, fake runners - Pre-commit hook failure documented: missing cp-work-claim-guard.sh
The existing CI workflow runs `npx tsc -p tsconfig.ci.json --noEmit` but tsconfig.ci.json did not exist, causing the type-check job to fail before any tests could run. This commit adds the minimum valid tsconfig.ci.json that extends the base tsconfig.json and includes the TypeScript source files under src/ without weakening strictness settings. Fixes baseline CI failure blocking every autonomous pull request.
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Closing as superseded per the PR body itself: contaminated with controller files from the #3 rewrite. The clean CI baseline has now landed on main via #8 (merged, b0a1867), which makes the missing-tsconfig case advisory — superseding this PR's purpose entirely. Original head SHA recorded in the body: ccca666 |
P0 #1: Path injection in execution-gate.js - add path.resolve + allowed-roots containment check in _getSearchDirs() P0 #2: Remove /api/health from PUBLIC_API_ROUTES + sanitize health endpoint (remove version, uptime, filesystem paths from response) P0 #3: Trust store path traversal in identity-enforcer.js - validate user-provided trustStorePath against ALLOWED_TRUST_STORE_ROOTS P0 #4: Race condition in lease-write.js moveFileWithLease - atomic claim-by-rename pattern instead of read-then-write P0 #5: Fix all hardcoded S:/SwarmMind paths to S:/SwarmMind Self-Optimizing Multi-Agent AI System across 50+ files
fix(ci): push auto-sync index to PR branch since main is protected
1 participant
SUPERSEDED — DO NOT MERGE
This PR is contaminated with controller files from the safe-autonomous-improvement-controller rewrite (PR #3).
Replaced by: Clean CI baseline PR from
origin/masterwith onlytsconfig.ci.json.Original Head SHA: ccca666
Status: Superseded. All CI changes will be handled in the clean replacement PR.