Skip to content

docs: refresh agent context, drop volatile counts, restore mirror - #47

Merged
prPMDev merged 1 commit into
masterfrom
docs/claude-md-refresh
Sep 5, 2026
Merged

docs: refresh agent context, drop volatile counts, restore mirror#47
prPMDev merged 1 commit into
masterfrom
docs/claude-md-refresh

Conversation

@prPMDev

@prPMDev prPMDev commented Sep 5, 2026

Copy link
Copy Markdown
Owner

The mirror was broken

Both files declare AGENTS.md canonical and CLAUDE.md its mirror. They had drifted: CLAUDE.md carried the entire Site deploy (GitHub Pages) section and AGENTS.md did not. An agent reading AGENTS.md got no warning about ?v= cache-busting or the required gtag and theme snippets, then breaks the site.

AGENTS.md is now generated from CLAUDE.md and differs only in the header comment. Verified identical below the headers.

Volatile counts do not belong in these files

The file already had the right principle, in a section titled "Where to find current state (live, not snapshot)":

Test baseline: node --test test/*.test.js should be green

No number. Then 70 lines earlier, the same command with # 124 tests, all should pass. The actual count is 142 and had been wrong for a while. Line 49 had the same problem with 700+ company verified registry, a number the weekly expansion changes by design.

Fixing the numbers just resets a clock. Removed them instead, and stated the rule explicitly so it does not come back.

Kept: the seven adapters and their names. That changes only when someone deliberately ships an adapter. User-facing counts in README.md and docs/index.html are a separate category with their own hundreds-boundary process, untouched.

Added

  • CONTRIBUTING.md and .github/workflows/ in the structure tree, both missing
  • The JD_INTEL_REGISTRY_URL gotcha: the library reads the registry from the published Pages URL, so a locally-added company returns 0 jobs until it merges. This currently burns anyone following CONTRIBUTING.md.
  • That the gate retries transient failures, and that a 429 is rate limiting rather than a dead board. Failed entries are dropped silently, so a future weekly run misreading 429s would quietly discard good companies.
  • CI exists now, so verification is not purely local

Three anti-patterns from today

  • Inferring an HTTP status by pattern-matching an error message. Workday messages embed the pod name, so /5\d\d/ reads "(ufp/wd503/Careers): 404" as a retryable 5xx.
  • Merging parallel agent output without reconciling against every output file. Agents write in passes; a mid-run merge silently dropped three verified entries today.
  • Stating a volatile count in an agent-context file.

Docs only. No code, no registry changes.

CLAUDE.md and AGENTS.md declare each other mirrors but had drifted:
AGENTS.md was missing the entire Site deploy section. AGENTS.md is now
regenerated from CLAUDE.md, differing only in the header block.

- Remove volatile counts (test total, registry size). The file already
  said "issues are the source of truth, not this file" while stating a
  test count that had gone stale. Give the command instead.
- Add CONTRIBUTING.md and .github/workflows to the structure tree
- Document the JD_INTEL_REGISTRY_URL local-testing gotcha
- Note that the gate retries transient failures, and that a 429 is not
  a dead board
- Three anti-patterns from today: status-by-message-regex, merging
  parallel agent output without reconciliation, stating volatile counts
@prPMDev
prPMDev merged commit c203552 into master Sep 5, 2026
4 checks passed
@prPMDev
prPMDev deleted the docs/claude-md-refresh branch September 5, 2026 21:05
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