Skip to content

docs: dependency DAG automation + workflows (#200) - #201

Merged
flyingrobots merged 8 commits into
mainfrom
echo/dependency-dags-automation
Jan 3, 2026
Merged

flyingrobots merged 8 commits into
mainfrom
echo/dependency-dags-automation

Conversation

@flyingrobots

@flyingrobots flyingrobots commented Jan 2, 2026 •

Copy link
Copy Markdown
Owner

Implements dependency DAG generation + automation, and documents contributor workflows.

Closes #200

Summary:

  • Adds issue/milestone dependency DAG generator (config-driven) and committed DOT/SVG artifacts.
  • Exposes generation via cargo xtask dags and Make targets; supports --snapshot-label none|rolling|YYYY-MM-DD.
  • Adds a scheduled GitHub Action that refreshes DAG outputs via PR only when outputs change (snapshot labels omitted to avoid churn).
  • Adds docs/workflows.md and links it from README + AGENTS.md.

Test Plan:

  • cargo xtask dags --snapshot-label none
  • cargo xtask dags --snapshot-label rolling --no-render

Adds issue/milestone DAG generator (Node + config), wraps it in cargo xtask, and documents contributor workflows. Includes a scheduled GH Action that refreshes DAG outputs via PR with snapshot labels omitted to avoid churn.
Adds docs/workflows.md (linked from README + AGENTS), documents DAG refresh issue linkage, and adds a helper script to open the tracking issue + PR safely.
Satisfies workspace clippy missing-docs policy for the xtask binary crate.
@coderabbitai

coderabbitai Bot commented Jan 2, 2026 •

Copy link
Copy Markdown
Contributor

Warning

Rate limit exceeded

@flyingrobots has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 2 minutes and 46 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between c893e50 and fa3d32a.

⛔ Files ignored due to path filters (5)
  • Cargo.lock is excluded by !**/*.lock
  • docs/assets/dags/issue-deps.dot is excluded by !**/*.dot
  • docs/assets/dags/issue-deps.svg is excluded by !**/*.svg
  • docs/assets/dags/milestone-deps.dot is excluded by !**/*.dot
  • docs/assets/dags/milestone-deps.svg is excluded by !**/*.svg
📒 Files selected for processing (17)
  • .cargo/config.toml
  • .github/workflows/refresh-dependency-dags.yml
  • .gitignore
  • AGENTS.md
  • Cargo.toml
  • Makefile
  • README.md
  • docs/assets/dags/deps-config.json
  • docs/decision-log.md
  • docs/dependency-dags.md
  • docs/docs-index.md
  • docs/execution-plan.md
  • docs/workflows.md
  • scripts/generate-dependency-dags.js
  • scripts/open_dependency_dags_pr.sh
  • xtask/Cargo.toml
  • xtask/src/main.rs
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch echo/dependency-dags-automation

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

Ensure the helper script writes 'Closes #<n>' without quoting so GitHub auto-closes the linked issue.
@flyingrobots

Copy link
Copy Markdown
Owner Author

@coderabbitai review

Merged latest main into this PR branch (including the PR-comment extractor and current CI/tooling baseline). Merge conflicts were resolved by preserving both PR-hygiene and DAG automation entries in docs/execution-plan.md / docs/decision-log.md and by keeping both .cache and Playwright artifact ignores in .gitignore.

Please review current head and post any actionables.

@coderabbitai

coderabbitai Bot commented Jan 2, 2026

Copy link
Copy Markdown
Contributor

@flyingrobots: I'll review the current head of PR #201 after the merge from main. I'll examine the DAG automation implementation, the resolved merge conflicts in the documentation files, and the overall changes to ensure everything is consistent and actionable.


🧠 Learnings used
Learnt from: CR
Repo: flyingrobots/echo PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-12-28T23:14:28.103Z
Learning: Keep `main` pristine. Feature work belongs on branches named `echo/<feature>` or `timeline/<experiment>`.

Learnt from: CR
Repo: flyingrobots/echo PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-12-28T23:14:28.103Z
Learning: When a PR touches non-doc code, update `docs/execution-plan.md` and `docs/decision-log.md` in the same PR.
✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@flyingrobots
flyingrobots merged commit 01f6b62 into main Jan 3, 2026
19 of 20 checks passed
@flyingrobots
flyingrobots deleted the echo/dependency-dags-automation branch January 3, 2026 01:50
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.

Automate dependency DAG refresh + document contributor workflows

1 participant