Skip to content

CTDL page: the validator is separate, not independent; say who wrote it - #173

Merged
ChelseaKR merged 2 commits into
mainfrom
fix/ctdl-validator-same-author
Sep 19, 2026
Merged

ChelseaKR merged 2 commits into
mainfrom
fix/ctdl-validator-same-author

Conversation

@ChelseaKR

Copy link
Copy Markdown
Owner

What does this change, and why?

/en/ctdl/ and /es/ctdl/ headed the validation findings "What an independent validator found". The validator is ctdl-validate, which has the same author as this site. The intro made it worse by introducing the tool right after a sentence about checks "written by the same hand as the export", which reads as if someone else wrote the tool.

The tool is separate in the way that matters for the check (its own copies of Credential Engine's schema, its own cited rules, no code shared with the exporter), and the page already said so. What it never said was who wrote it.

Changes:

  • Heading: "What a separate validator found" / "Qué encontró un validador aparte"
  • Opening paragraph: "what a separate validator found when it was pointed at the result" / "qué encontró un validador aparte al revisar el resultado"
  • Validation intro gains one sentence: "It has the same author as this site, so it is a second reading of the schema and not an outside review." / "Tiene la misma autora que este sitio, así que es una segunda lectura del esquema y no una revisión externa."
  • Two docstrings (cli.py, ctdl/validate.py) reworded. CHANGELOG.md history untouched; one new entry under Fixed.

Found while fact-checking an essay: a checker opened the linked validation statement, saw the tool's source was github.com/ChelseaKR/ctdl-validate, and flagged "independent" as a major issue. The essay now says "a separate validator I also wrote"; this makes the page agree.

Please look at before merging: the two new Spanish sentences are machine-drafted, like the rest of the Spanish (the site's notice covers them). "la misma autora" is feminine on purpose.


Checks

  • uv run mypy (strict; src, scripts, tests): no issues in 113 source files
  • ruff check and ruff format --check pass; all pre-commit and pre-push hooks pass
  • tsc --noEmit passes
  • vitest run lib: 426 passed, 9 skipped, 0 failed tests. Two test files (lib/alternates.test.ts, lib/site.test.ts) could not load in a fresh worktree because web/public/data/search-index.json is generated and was absent; unrelated to this change, CI builds it
  • Full make verify and cd web && npm run verify (contrast, build, axe) not run locally; leaving to CI
  • Copy only: no figure, colour, or layout changes, so the display and interface checklists do not apply

AI disclosure: drafted with Claude Code at the repository owner's request. Owner review required before merge.

🤖 Generated with Claude Code

ChelseaKR and others added 2 commits September 18, 2026 21:31
ctdl-validate has the same author as this site. The page headed its findings
"What an independent validator found" and introduced the tool right after a
sentence about checks "written by the same hand as the export", which read as
if someone else had written it.

Heading and opening paragraph now say "separate". The validation intro gains one
sentence: it has the same author as this site, so it is a second reading of the
schema and not an outside review. Spanish strings changed to match. Two
docstrings that used the phrase are reworded. CHANGELOG history is left as it was.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@ChelseaKR
ChelseaKR merged commit 39c45e4 into main Sep 19, 2026
4 checks passed
@ChelseaKR
ChelseaKR deleted the fix/ctdl-validator-same-author branch September 19, 2026 04:56
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