Skip to content

fix: harden LLM docs worker handover checks - #13

Merged
JimCollinson merged 2 commits into
mainfrom
fix/handover-review-hardening
Jun 26, 2026
Merged

fix: harden LLM docs worker handover checks#13
JimCollinson merged 2 commits into
mainfrom
fix/handover-review-hardening

Conversation

@dirvine

@dirvine dirvine commented Jun 25, 2026

Copy link
Copy Markdown
Member

Summary

  • fix the machine-readable Who We Are links to point at the served Markdown path
  • restrict Worker GitHub proxying to GET/HEAD and add method coverage tests
  • run the full Worker check suite in manual preview/production deploy workflows before deploying

Verification

  • npm run check
  • python3 scripts/adr-governance.py

Raised after Hermes review of repo maintainability and handover readiness.

@JimCollinson

Copy link
Copy Markdown
Member

Thanks, this looks directionally good.

One requested tweak before merge: overview.md still links [Who We Are](/who-we-are). Since this PR updates the machine-readable indexes to use https://autonomi.com/who-we-are.md, can you update overview.md to link to /who-we-are.md as well?

On the Worker method change: I’m comfortable treating the GET/HEAD-only proxying as implementation hardening, not a new ADR, as long as we’re explicit that non-GET/HEAD requests are outside the docs-serving contract and intentionally fall through.

@JimCollinson

Copy link
Copy Markdown
Member

Made those small tweaks!

@JimCollinson
JimCollinson merged commit 56967ec into main Jun 26, 2026
1 check passed
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.

2 participants