Skip to content

Remove leftovers from the image-docs pipeline - #4102

Merged
matthewhelmke merged 2 commits into
mainfrom
matthewhelmke/docs-138-pipeline-leftovers
Sep 30, 2026
Merged

matthewhelmke merged 2 commits into
mainfrom
matthewhelmke/docs-138-pipeline-leftovers

Conversation

@matthewhelmke

Copy link
Copy Markdown
Collaborator

Cleanup after #4099, which made the AI docs bundle compile from this repository alone. None of these changes alters what the build does.

Changes

  • Drop the httpx pin from scripts/mcp-requirements.txt. The pin served the registry client that Remove the AI Docs MCP server's frozen image tools; restore dfc mappings #4099 removed, and nothing imports httpx now. mcp 2.2 depends on httpx2, a separate package, so the pin only installed an unused library.
  • Delete .github/workflows/templates/. Its three files were blueprints for the images-private and courses exports to GCS, which nothing reads anymore. Actions never ran them.
  • Rewrite docs/automated-docs-setup.md. The guide still described four sources, those templates, GCS downloads, and a local test that needed placeholder sibling directories. It now covers the two sources, the scripts/ trigger, the package catalog, and a local test that runs from a plain checkout.

Deliberately not changed

The edu export's GCS upload and the compile workflow's compiled/ upload stay as they are. Neither has a known reader, but nobody can check the bucket's access logs. DOCS-218, due 2026-10-28, revisits both, once the 2026-09-30 MCP changes have settled and if ownership or readers become known.

Testing

  • In a fresh virtual environment built from the new requirements, httpx isn't installed, and pytest scripts/test_mcp_server.py scripts/test_mcp_transport.py scripts/test_compile_docs.py passes (45 tests).
  • Ran every command in the rewritten guide as written. The compile logs DFC mappings added, and the catalog builds with 683 package mappings and 397 image mappings.
  • grep finds no remaining reference to .github/workflows/templates/.

Refs DOCS-138, DOCS-218

🤖 Generated with Claude Code


Created in collaboration with Claude Code running Opus 5.5 on 2026-09-30.

matthewhelmke and others added 2 commits September 30, 2026 10:22
httpx was pinned for the registry client behind check_image_freshness,
which edu#4099 removed. Nothing imports it now, and mcp 2.2 depends on
httpx2, a separate package, so the pin only installed an unused
library. The unit tests pass in a fresh environment without it.

Refs DOCS-138

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The templates in .github/workflows/templates/ were blueprints for the
images-private and courses exports to GCS. Since edu#4099 the AI docs
bundle compiles from this repository alone, so nothing reads those
exports, and Actions never ran the templates.

docs/automated-docs-setup.md described four sources, the templates, GCS
downloads, and a local test that needed placeholder sibling directories.
It now describes the two sources, the scripts/ trigger, the package
catalog, and a local test that runs from a plain checkout. Every
command in it was run as written.

Refs DOCS-138

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@matthewhelmke
matthewhelmke requested a review from a team as a code owner September 30, 2026 15:24
@vercel

vercel Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
chainguard-docs-preview Ready Ready Preview Sep 30, 2026 3:24pm UTC

Request Review

@matthewhelmke
matthewhelmke requested a review from a team as a code owner September 30, 2026 15:24
@netlify

netlify Bot commented Sep 30, 2026 •

Copy link
Copy Markdown

✅ Deploy Preview for ornate-narwhal-088216 ready!

Name Link
🔨 Latest commit a688f6e
🔍 Latest deploy log https://app.netlify.com/projects/ornate-narwhal-088216/deploys/6abd29971c4add0008f08ac5
😎 Deploy Preview https://deploy-preview-4102--ornate-narwhal-088216.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@matthewhelmke
matthewhelmke merged commit 367cf77 into main Sep 30, 2026
21 checks passed
@matthewhelmke
matthewhelmke deleted the matthewhelmke/docs-138-pipeline-leftovers branch September 30, 2026 15:34

This branch was successfully deployed

1 active deployment
Preview — a688f6e0 Deployed Sep 30, 2026 by vercel[bot]
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