Skip to content

Suite: remediación de seguridad + empaquetado multi-editor#1

Merged
PRojas01 merged 1 commit into
masterfrom
suite-security-and-surfaces
Jul 20, 2026
Merged

Suite: remediación de seguridad + empaquetado multi-editor#1
PRojas01 merged 1 commit into
masterfrom
suite-security-and-surfaces

Conversation

@PRojas01

Copy link
Copy Markdown
Owner

Integra la consolidación de la suite. Seguridad: cierra el bypass de auto-aprobación (MCP/API/CLI exigen token humano), hooks fail-closed. 120/120 tests. Superficies: plugin Claude Code, skill, .codex, opencode.json, extensión VS Code (pestaña), CI de release. Revisado en 3 rondas red team + verificado directamente. Ver los archivos modificados de src/ para los fixes de seguridad.

… reverts

Aplica SOLO el aporte de la consolidación sobre la base actual del repo:
Seguridad (verificada 3 rondas red team):
- src/human-approval.ts (nuevo): token humano fuera de banda.
- api.ts/cli.ts/gates.ts/mcp.ts/policy.ts: aprobaciones (approve_change, session:approval:grant, gate:approve) exigen token humano en MCP/API/CLI. Fin del auto-aprobado agencial.
- hooks.ts: fail-closed, sin sugerir --no-verify.
- Tests: 118/118.
Superficies: .claude-plugin (plugin Claude Code), .claude/skills, .codex, opencode.json, extension/ (VS Code, pestaña dentro del editor), INSTALL.
No toca release.yml/package.json/README/CI upstream.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@PRojas01
PRojas01 force-pushed the suite-security-and-surfaces branch from 22300f9 to c9f1b1d Compare July 19, 2026 22:27
@PRojas01
PRojas01 merged commit f411fb7 into master Jul 20, 2026
0 of 3 checks passed
@PRojas01
PRojas01 deleted the suite-security-and-surfaces branch July 20, 2026 22:10
PRojas01 added a commit that referenced this pull request Jul 20, 2026
…p v2.1.1

PR #1 (security + multi-editor surfaces) shipped incomplete: src/session-approval.ts
was missing entirely (broke the build), and injector.ts was left on a stale
version incompatible with cli.ts (writeInjectionFiles returned string[] instead
of WriteResult[], losing custom-content preservation). Both fixed from the
consolidation source. Bump to 2.1.1, CHANGELOG documents the human-approval
security fix, README updated (extension section, control count). 118/118
tests passing, clean build.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant