Skip to content

feat: add scratch codegen executor registry#40

Closed
wesleysimplicio wants to merge 1 commit into
masterfrom
codex/finalize-pr-#39
Closed

feat: add scratch codegen executor registry#40
wesleysimplicio wants to merge 1 commit into
masterfrom
codex/finalize-pr-#39

Conversation

@wesleysimplicio
Copy link
Copy Markdown
Owner

Motivation

  • Permitir execução determinística/mechanical de tarefas scratch sem depender do LLM, fornecendo um caminho de execução previsível e testável antes do fluxo de LLM e mantendo o fallback para LLM quando necessário.

Description

  • Adiciona contratos e API do registry em simplicio/scratch/codegen: types.py (CodegenResult, TaskExecutor), registry.py (register_executor, registered_executors, try_execute) e __init__.py exportando a API.
  • Integra o fluxo do registry em simplicio/scratch/executor.py chamando try_execute antes do caminho LLM, propagando codegen logs e preservando a semântica de fallback_to_llm e o modo de stub quando SIMPLICIO_MODEL não está definido.
  • Adiciona tests/python/test_scratch_codegen.py cobrindo registry vazio, seleção do primeiro executor compatível, execução bem-sucedida sem modelo, falha sem fallback e comportamento de fallback.
  • Nenhuma dependência externa nova foi adicionada (nenhum libcst/ts-morph etc.); mudanças são uma fundação de plumbing para executores mecânicos.

Testing

  • Rodei python -m pytest tests/python -q -p no:cacheprovider e os testes Python relevantes passaram (os novos testes de scratch_codegen executaram e passaram; no conjunto completo apareceram 119 passed, 1 skipped).
  • Rodei node scripts/lint.js e o linter concluiu sem erros (apareceram avisos de trailing whitespace e avisos de ferramentas ausentes, mas sem falhas).
  • Rodei npx playwright test --reporter=list,html (após npx playwright install e npx playwright install-deps), e os cenários E2E relevantes passaram (E2E: 6 passed, 27 skipped na run desta máquina de verificação).
  • npm test não está disponível neste repositório porque package.json não define um script test, portanto não foi executado aqui.

Codex Task

@wesleysimplicio
Copy link
Copy Markdown
Owner Author

Closing as duplicate/stale. PR #39 now contains this foundation slice plus the LibCST/ts-morph executors, scratch evidence harnesses, pilot stacks, planner cache, cache telemetry, and SkillOpt inline updates.

@wesleysimplicio wesleysimplicio deleted the codex/finalize-pr-#39 branch May 31, 2026 10:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant