Skip to content

docs(bazel): documentation, observability, and #1 close-readiness evidence (#3) - #428

Merged
DecisionNerd merged 1 commit into
mainfrom
docs/3-bazel-close-readiness
Aug 7, 2026
Merged

DecisionNerd merged 1 commit into
mainfrom
docs/3-bazel-close-readiness

Conversation

@DecisionNerd

@DecisionNerd DecisionNerd commented Aug 7, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Closes #3

Test plan

Made with Cursor


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Note

Add Bazel developer guide and #1 close-readiness evidence map to docs

  • Adds bazel.md as the primary developer guide covering installation, local commands, extending the build graph, cache metrics, CI/release runbooks, and security constraints.
  • Adds bazel-migration-ac-evidence.md as a checked-in close-readiness map tying build: migrate CI builds and tests to Bazel with Blacksmith caching #1 acceptance criteria to child issues, PRs, and merge SHAs.
  • Updates OBSERVABILITY.md with Bazel CI correctness and remote-cache health as observable outcomes, including dashboard/alert guidance.
  • Updates release-process.md to explicitly state that Bazel owns CI Rust compilation and that publish credentials must never appear in cacheable Bazel actions.
  • Cross-links all existing migration docs (bootstrap, cutover, parity, orchestration) and ARCHITECTURE.md to the new guides, replacing outdated 'Next' sections.

Macroscope summarized 35e31b4.

)

Consolidate M2 documentation, observability paths, and supply-chain evidence
so every #1 AC has a child-issue pointer for gate #2.

Co-authored-by: Cursor <cursoragent@cursor.com>
@coderabbitai

coderabbitai Bot commented Aug 7, 2026

Copy link
Copy Markdown

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (11)
  • docs/development/bazel-bootstrap.md is excluded by !**/*.md, !**/docs/**
  • docs/development/bazel-migration-ac-evidence.md is excluded by !**/*.md, !**/docs/**
  • docs/development/bazel-migration-cutover.md is excluded by !**/*.md, !**/docs/**
  • docs/development/bazel-migration-orchestration.md is excluded by !**/*.md, !**/docs/**
  • docs/development/bazel-migration-parity.md is excluded by !**/*.md, !**/docs/**
  • docs/development/bazel.md is excluded by !**/*.md, !**/docs/**
  • docs/development/contributing.md is excluded by !**/*.md, !**/docs/**
  • docs/development/release-process.md is excluded by !**/*.md, !**/docs/**
  • docs/engineering/ARCHITECTURE.md is excluded by !**/*.md, !**/docs/**
  • docs/engineering/OBSERVABILITY.md is excluded by !**/*.md, !**/docs/**
  • docs/engineering/README.md is excluded by !**/*.md, !**/docs/**

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 57b18df3-a85f-4e24-a7bb-4edaadd8920a

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Comment @coderabbitai help to get the list of available commands.

@github-actions github-actions Bot added documentation Improvements or additions to documentation release:none No release note or version impact labels Aug 7, 2026
@DecisionNerd
DecisionNerd merged commit 910af15 into main Aug 7, 2026
20 checks passed
@DecisionNerd
DecisionNerd deleted the docs/3-bazel-close-readiness branch August 7, 2026 06:42
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 release:none No release note or version impact

Projects

None yet

Development

Successfully merging this pull request may close these issues.

bazel: documentation, observability, and #1 close-readiness evidence

1 participant