aboutsummaryrefslogtreecommitdiffstats
path: root/docs/plans/2026-09-09-contacts.md
AgeCommit message (Collapse)AuthorFilesLines
38 hoursdocs: correct the Accept header and the dependency claimDanilo M.1-1/+1
The spec named a strict Accept: application/rdap+json while the plan, and so the code, sends application/rdap+json with application/json at q=0.9. A spec review caught the disagreement. The wider header is the right behaviour, since some registries serve plain application/json and a strict header would lose a contact that was published, so the spec moves to match the code and now records why. Both documents also claimed requirements.txt stays empty. There is no requirements.txt in this repository and there never has been, so the claim was true only vacuously. They now say the tool declares no dependencies, which is the thing actually being promised. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Wrfqr2xqQfhtXCscU7zrdz
38 hoursdocs: amend the contacts plan after task 1 to 5Danilo M.1-4/+26
Two corrections found while executing it. The offline proof in task 9 patched socket.socket itself, which breaks the ssl module at import time and produces false failures unrelated to network use. It now patches socket.socket.connect, which is the right target. Task 12 gains a step moving the imports in rdap.py to the top of the file. The module was built one task at a time, so several imports landed mid-file after function definitions. Doing it in the documentation pass keeps it out of the diff that contacts.py builds on. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Wrfqr2xqQfhtXCscU7zrdz
38 hoursdocs: plan the contacts implementationDanilo M.1-0/+1985
Twelve tasks, TDD throughout, building bottom up: transport, bootstrap cache, server selection, jCard extraction, querying with the label walk, the worklist, resolution, the subcommand, an offline proof, two sweeps and the documentation. The code in the plan was assembled and run before committing it, against the plan's own tests, 44 of which pass. That found two defects in the plan itself: a parameter that shadowed the cache_dir function and forced a globals() lookup, now renamed to cache_root, and a hand-rolled fake request in the redirect test that lacked the attributes HTTPRedirectHandler reads, now a real urllib Request. Also drops the em dashes from the Documents list in AGENTS.md, which the no-em-dash writing rule covers and which the contacts entry had matched. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Wrfqr2xqQfhtXCscU7zrdz