Fix link to AI Policy in CONTRIBUTING.md - #7576
Merged
Merged
Conversation
nateprewitt
requested changes
Jul 9, 2026
| Before opening any issues or proposing any pull requests, please read | ||
| our [Contributor's Guide](https://requests.readthedocs.io/en/latest/dev/contributing/) | ||
| as well as our [AI Policy](./AI_POLICY.md). | ||
| as well as our [AI Policy](./.github/AI_POLICY.md). |
Member
There was a problem hiding this comment.
We may need a fully qualified link here. The reason it's broken is because it's rendered at two different points.
It's correct if you're viewing CONTRIBUTING.md directly, but broken for when you click the link for Contributing on the repo.
Contributor
Author
There was a problem hiding this comment.
Ah well I just checked it if it works on the tab, not by viewing the file on repo. Good catch.
yamac-kurtulus
commented
Jul 9, 2026
| Before opening any issues or proposing any pull requests, please read | ||
| our [Contributor's Guide](https://requests.readthedocs.io/en/latest/dev/contributing/) | ||
| as well as our [AI Policy](./.github/AI_POLICY.md). | ||
| as well as our [AI Policy](/.github/AI_POLICY.md). |
Contributor
Author
There was a problem hiding this comment.
Tested on my repo, works both from the tab, and the file itself, turns out absolute path from root is sufficient, not a full path.
nateprewitt
approved these changes
Jul 9, 2026
nateprewitt
left a comment
nateprewitt
left a comment
Member
There was a problem hiding this comment.
Alright, I think this'll work. Thanks, @yamac-kurtulus!
luisfelp07
added a commit
to marianocho/hack2l
that referenced
this pull request
Aug 10, 2026
Medido em 10 PRs de Flask, Django, httpx, Gin, Next.js e Requests.
acusacoes com arbitro preenchido 94
arbitro do vocabulario do Hack2L 93 (99%)
o 94o 'R1 R2 R3 R4 AC1 AC2 AC3 AC4 AC5'
-- a lista inteira do prompt, colada
Fora do desafio a taxa real de arbitro e' ZERO. Os 45% que eu tinha
comemorado como 'acima do piso' mediam contaminacao, nao cobertura.
Explica tambem por que a lente de prd nunca ficou vazia em 10 PRs: ela
sempre tem criterio para conferir porque os criterios estao dentro dela.
Nao lia o repositorio, recitava o desafio.
CONTROLE NEGATIVO (psf/requests#7576, um link de markdown, 11 acusacoes
ao advogado com read_file e grep):
REFUTADO 7 (64%) · INCONCLUSIVO 2 (18%) · PROVADO 2 (18%)
US$ 0,61 e 328s para limpar um PR de documentacao
Sete refutados e' a boa noticia: a divisao promotor/advogado funciona.
Os dois sobreviventes tem causas diferentes:
1. erro de raciocinio com a direcao invertida -- alegou que caminho
absoluto em markdown vira URL de dominio (falso; o GitHub resolve da
raiz do repo) e acusou o CONSERTO de ser o defeito
2. o PRD do Hack2L aplicado ao psf/requests: 'nenhum requisito R1-R4 ou
AC1-AC5 pode ser validado por esta mudanca'. O advogado provou uma
verdade trivial sobre criterios de outro projeto.
CONSERTO: arbitro vira citacao com procedencia do repo sob revisao
({regra, onde}), nao lista fixa. Sem fonte, null -- que e' a resposta
honesta. Consequencia seria: com null honesto a R1 rebaixa quase tudo
para SUSPEITA, e o produto passa a precisar de uma segunda via para
severidade alta que nao dependa de arbitro.
Tambem consertado neste commit: o relatorio somava INCONCLUSIVO com
REFUTADO e dizia 'refutou tudo' quando nenhuma ferramenta tinha
funcionado. Absolvicao falsa no proprio medidor.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
luisfelp07
added a commit
to marianocho/hack2l
that referenced
this pull request
Aug 17, 2026
O bloqueador de adocao numero 1 da fila. Ate' agora o alvo era BRANCH_PR fixo
apontando para pasta local; aponta-se o produto para um PR do GitHub e ele nao
sabia o que fazer.
py -3.12 revisa_pr.py https://github.com/dono/repo/pull/123
Conferido de ponta a ponta contra psf/requests#7576: resolve, clona, busca os
dois commits e os DOIS worktrees montam nos SHAs certos.
🚨 O BASE E' O MERGE-BASE, E ISSO NAO E' DETALHE
`pulls/{n}` devolve `base.sha` = o topo do ramo alvo AGORA. Se a main andou
depois de o PR abrir, esse commit carrega mudanca que nao e' do PR -- e a prova
diferencial ("passa no base, falha no head") passaria a medir o PR SOMADO ao
que entrou no meio. Falso positivo com cara de prova, e mudo.
O commit certo vem do endpoint `compare`, campo `merge_base_commit`. Uma
chamada a mais. E nao da' para calcular localmente: `git merge-base` precisa da
historia dos dois lados e o clone e' raso de proposito. Quando os dois diferem,
`base_deslocou` fica no artefato -- quem auditar amanha precisa saber contra o
que a prova rodou.
🚨 OS DOIS COMMITS, NAO SO' O HEAD
O `controle_negativo.py` buscava so' o head e apontava o base para o mesmo sha
("nao usamos base aqui"). Para revisar de verdade os dois sao obrigatorios: sem
base nao ha prova diferencial, que e' a unica via que assina PROVADO junto com
o arbitro.
E o fetch e' CONFERIDO: servidor com allowReachableSHA1InWant desligado recusa
sha solto e ainda pode sair 0. Sem `cat-file -e`, o worktree seguinte montaria
outra coisa e o artefato registraria o commit PEDIDO -- mesma familia de falso
negativo mudo que o _garante_worktree existe para impedir do outro lado.
## Em processo separado, de proposito
O config resolve PROJETO, contas, rota de login, banco e layout NA IMPORTACAO,
a partir de CHALLENGE_REPO. Trocar cfg.DESAFIO depois deixaria metade da
configuracao no projeto anterior -- o item 4 dos cinco chumbados de 15/08, em
que o PRE-VOO PASSOU VERDE enquanto a rodada revisava um repo e conversava com
o app de outro. Ambiente primeiro, importacao depois. Mesmo desenho do
roda_bancada.py.
## O que ele diz ANTES de gastar
Sem veredito.yml no repo revisado, avisa que a rodada tera' so' read_file e
grep, e que a R2 limita a severidade a MEDIA. Descobrir isso no parecer todo em
MEDIA seria descobrir tarde.
Com veredito.yml na raiz -- arquivo do cliente, no repositorio do cliente, como
um Dockerfile -- `projeto.caminho()` ja o encontra e as ferramentas todas
voltam. Nada a fazer ali: a busca existe desde 14/08.
⚠️ O token vai em `http.extraheader`, NUNCA no remote: `git remote -v` e o
.git/config guardariam a credencial em texto no disco. Tem teste.
18 testes, sem rede. 475 no total.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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
2 participants
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.
The AI Policy is not accessible with a relative link inside .github folder, this PR fixes it.