Skip to content

docs: establish repository knowledge system - #128

Merged
dannysteenman merged 1 commit into
mainfrom
codex/repo-knowledge-system
Jul 21, 2026
Merged

docs: establish repository knowledge system#128
dannysteenman merged 1 commit into
mainfrom
codex/repo-knowledge-system

Conversation

@dannysteenman

Copy link
Copy Markdown
Member

Summary

  • Rework AGENTS.md into a concise table of contents and add docs/README.md as the canonical knowledge map.
  • Consolidate stale review and Turborepo pages into current guides and references for development, release, commands, generated files, tests, configuration, findings, and rule IDs.
  • Add a mechanically enforced documentation graph with 14 CLI contract tests and a dedicated CI job.
  • Record the unresolved stable-public-ID versus contiguous-per-service-ID contract without changing rule IDs or runtime behavior.

Diagram

flowchart LR
  A["AGENTS.md"] --> B["docs/README.md"]
  C["README.md"] --> B
  B --> D["Guides"]
  B --> E["Reference"]
  B --> F["Package AGENTS.md files"]
  G["CI docs checker"] --> A
  G --> B
  G --> D
  G --> E
Loading

Scope

  • cloudburn (cli)
  • @cloudburn/sdk
  • @cloudburn/rules
  • docs/community files

Release Notes

  • Added a .changeset/*.md file for published package changes
  • No published package changes in this PR

Verification

  • pnpm lint
  • pnpm typecheck
  • pnpm test
  • pnpm build
  • pnpm verify
  • pnpm docs:test (14 tests)
  • pnpm docs:check
  • git diff --check

Boundary Checks

  • No engine/parser/provider logic added to @cloudburn/rules
  • CLI delegates scan logic to SDK
  • README/CONTRIBUTING/docs updated when behavior changed

Related Issues

No directly related open issue was found.

@dannysteenman dannysteenman added the documentation Improvements or additions to documentation label Jul 21, 2026 — with ChatGPT Codex Connector
@dannysteenman
dannysteenman requested a review from axonstone July 21, 2026 11:57
@axonstone

Copy link
Copy Markdown
Contributor

roborev: Combined Review (9827a67)

Verdict: Fail — the unresolved rule-ID contract risks silently changing user configuration semantics.

High

  • Rule-ID compatibility is internally contradictorydocs/reference/rule-ids.md treats IDs as public configuration/result references while requiring renumbering to preserve gap-free sequences. The release guide prohibits major changesets and provides no migration or alias mechanism. Removing or reordering a rule could therefore make an existing enabled-rules entry silently select a different policy.
    • Recommended fix: Use immutable IDs with permitted gaps or tombstones and update the metadata test. Otherwise, define versioning, aliases, deprecation, migration diagnostics, and a release process for breaking renumbering.

Reviewers: 3 done | Synthesis: codex, 12s | Total: 5m35s

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9827a67bc8

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread scripts/check-docs.mjs
@dannysteenman
dannysteenman merged commit 0d8920a into main Jul 21, 2026
8 checks passed
@dannysteenman
dannysteenman deleted the codex/repo-knowledge-system branch July 21, 2026 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants