Repository navigation
chore: remover o harness de Vercel Sandbox do fork - #26
Merged
Merged
Conversation
Este fork não tem credenciais Vercel nem intenção de usar o Sandbox, mas o AGENTS.md declarava `pnpm test:sandbox` como o gate antes de entregar qualquer mudança. O efeito prático era mandar todo agente para um gate inexecutável e deixar o gate real — as duas lanes locais — como um "fallback". Remove os onze arquivos do harness, as duas entradas de script do package.json e as referências em AGENTS.md, README.md, no README do test262 e nos dois planos sob .red/code/plans. O gate declarado passa a ser as lanes plain e sanitizada locais. O harness é código mantido em upstream/main, então esta remoção é uma divergência deliberada do fork e vai conflitar nas próximas sincronizações; a resolução é descartar o harness de novo. SCRIPTC_PORTABLE_ONLY continua nos testes que o consultam, mas nada o define mais: era o script de Sandbox que o ligava. Os contratos nativos de host passam a rodar sempre localmente.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Este fork não tem credenciais Vercel nem intenção de usar o Sandbox, mas o
AGENTS.mddeclaravapnpm test:sandboxcomo "the bar before shipping any change". O efeito prático era mandar todo agente para um gate inexecutável e rebaixar o gate real — as duas lanes locais — à condição de fallback. Isso acabou de acontecer numa frente de trabalho.O que sai
Dockerfile.sandbox,scripts/sandbox-{bootstrap.sh,command,config,image,platform,test}e os quatrotests/harness/sandbox-*.test.ts.package.json(test:sandbox,test:sandbox:image).AGENTS.md,README.md,tests/test262/README.mde nos dois planos sob.red/code/plans.O que passa a valer
O gate declarado é as duas lanes locais, promovidas ao bloco de comandos do
AGENTS.md:Duas consequências que ficam registradas
Divergência deliberada do upstream. O harness é código mantido em
upstream/main— entrou aqui pela integração do v0.1.6. Esta remoção vai conflitar nas próximas sincronizações, e a resolução é descartar o harness de novo. A decisão foi tomada com esse custo na mesa.SCRIPTC_PORTABLE_ONLYficou inerte. Cinco testes ainda o consultam, mas era o script de Sandbox que o definia; nada mais o liga. Na prática os contratos nativos de host passam a rodar sempre localmente, que é o comportamento desejado nesta máquina.Verificação
pnpm -r buildlimpo,package.jsonválido, e nenhuma referência ativa restante ao harness fora de documentos históricos sob.red/researchese.red/checkpoints, que registram o que era verdade na época e foram preservados.🤖 Generated with Claude Code