Skip to content

docs: refresh dx-stack guidance - #359

Merged
GregHolmes merged 2 commits into
mainfrom
gh/remove-stale-dx-stack-guidance
Oct 23, 2026
Merged

GregHolmes merged 2 commits into
mainfrom
gh/remove-stale-dx-stack-guidance

Conversation

@GregHolmes

Copy link
Copy Markdown
Contributor

Updates DX stack guidance to the current dx-stack files and structured registries.

Validation: git diff --check.

GregHolmes and others added 2 commits October 23, 2026 11:55
…document-change skill

docs/architecture.md in dx-stack is the design essay, not the port map or auth flows; those live in AGENTS.md and docs/auth.md. Also name the skill that replaced the deleted per-repo hook.

@dg-coreylweathers dg-coreylweathers left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Content reviewed and verified; two fixes pushed to this branch in f794791. Approval is a separate click.

What I verified against a fresh clone of dx-stack main (4b5af9e), run in a throwaway container: every path in the table exists; AGENTS.md has the port map (L515), cross-service (L461) and deployment (L562) sections; docs/auth.md has the client table (L68) and schema sections (L354/376); hooks/check-cross-stack.sh was deleted in dx-stack 76b8bd7.

What f794791 changes:

  1. Rule 3 pointed at docs/architecture.md for the port map, auth flows, service-to-service and environment matrix. That file is the neurosymbolic design essay and holds none of them; they live in AGENTS.md (port map, cross-service, environments) and docs/auth.md (auth flows, L117). Now reads: Read AGENTS.md (port map, cross-service architecture, environments, deployment) and docs/auth.md (auth flows) in your dx-stack checkout for the full stack context.
  2. Rule 2 never named what replaced the hook. dx-stack AGENTS.md (L15-21) says docs are kept current by the dx-document-change skill via the dx-stack MCP, and that skill describes itself as the replacement for the per-repo check-cross-stack.sh hook. Added one sentence saying so, with the table kept as the manual fallback.

Both original lines are already merged in dx-id's CLAUDE.md, so dx-id needs the same two edits.

Nit, not changed: bare seed.ts in the Redirect URI row is dx-id/apps/id/src/db/seed.ts, not a dx-stack file.

@dg-coreylweathers dg-coreylweathers left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved. Verified against dx-stack main and the two should-fixes are on this branch; see the comment above.

@GregHolmes
GregHolmes merged commit 0dbe949 into main Oct 23, 2026
@GregHolmes
GregHolmes deleted the gh/remove-stale-dx-stack-guidance branch October 23, 2026 12:14
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