Skip to content

#306 — fix: examples/cute-dbt-pr-review.yml missing the report verb — exits 2 since #100 - #309

Merged
cmbays merged 1 commit into
mainfrom
infra-306-example-yml-report-verb
Jun 12, 2026
Merged

cmbays merged 1 commit into
mainfrom
infra-306-example-yml-report-verb

Conversation

@cmbays

@cmbays cmbays commented Jun 12, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Since #100 made the subcommand required, the committed consumer example workflows still carried the pre-#100 flat surface — a bare cute-dbt --manifest ... that now exits 2 (clap usage error). This adds the report verb.

The AC's sweep surfaced a second drifted example beyond the one named in the issue, fixed in the same pass (same bug class; the sweep must come back clean):

  • .github/workflows/examples/cute-dbt-pr-review.yml line 143 (pr-diff arm — the one named in the issue)
  • .github/workflows/examples/cute-dbt-report-preview.yml line 81 (baseline arm)

Evidence

  • actionlint exit 0 on both touched files
  • Sweep rg -n 'cute-dbt \\$|cute-dbt --manifest' book .github README.md → zero matches (was 2)
  • Broader rg 'cute-dbt --' sweep: every remaining hit is a verbed cargo run -- report|explore, an install command, or cute-dbt --help — no verbless invocation remains in any committed doc/example
  • No Rust code touched; goldens unaffected

Context

Found during #294 shape research (research-294/sweep-cli-invocation-contract.md §5): the recipe markdown (book/src/recipes/github-actions-pr-review.md) was already correct — only the yml examples drifted.

Closes #306

🤖 Generated with Claude Code


Open in Stage

Summary by CodeRabbit

  • Chores
    • Updated GitHub Actions workflows to improve CLI command invocation for the reporting tool. These internal workflow updates ensure consistent command execution across automated processes.

Since #100 made the subcommand required, the committed consumer
examples' bare 'cute-dbt --manifest ...' invocations exit 2 (clap
usage error). The AC sweep surfaced a second drifted example beyond
the one named in the issue:

- .github/workflows/examples/cute-dbt-pr-review.yml (pr-diff arm)
- .github/workflows/examples/cute-dbt-report-preview.yml (baseline arm)

Both now invoke 'cute-dbt report'. actionlint clean on both; the
verbless-invocation sweep (rg 'cute-dbt \\$|cute-dbt --manifest'
over book/.github/README) now returns zero matches.

Found during #294 shape research
(research-294/sweep-cli-invocation-contract.md section 5).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@coderabbitai

coderabbitai Bot commented Jun 12, 2026 •

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: fe3ddea4-03a9-4804-bb9c-d72c284735a7

📥 Commits

Reviewing files that changed from the base of the PR and between 90c320b and 9227b44.

📒 Files selected for processing (2)
  • .github/workflows/examples/cute-dbt-pr-review.yml
  • .github/workflows/examples/cute-dbt-report-preview.yml

📝 Walkthrough

Walkthrough

Two GitHub Actions example workflows are updated to invoke the cute-dbt CLI with the explicit report subcommand. The pr-review workflow line 143 and report-preview workflow lines 81–83 both change from a generic cute-dbt prefix to the correct cute-dbt report command form, aligning with the required subcommand structure.

Changes

CLI Invocation Updates

Layer / File(s) Summary
Report subcommand CLI fixes
.github/workflows/examples/cute-dbt-pr-review.yml, .github/workflows/examples/cute-dbt-report-preview.yml
Both example workflows are updated to invoke cute-dbt report with the explicit report subcommand instead of the generic cute-dbt prefix, reflecting the required verb structure.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • breezy-bays-labs/cute-dbt#72: Introduced the cute-dbt report-preview workflow and its report-rendering command structure that is being corrected here.

Poem

🐰 A verb was lost in workflows two,
cute-dbt report now shines anew.
With subcommands sharp and aligned,
The CLI invocations are refined! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly identifies the main fix: adding the missing 'report' verb to example workflow files to comply with requirements from #100.
Linked Issues check ✅ Passed The PR fully addresses all acceptance criteria from #306: adds 'report' verb to example workflows, actionlint verification passed, and sweep confirmed no other verbless invocations remain.
Out of Scope Changes check ✅ Passed All changes are in-scope: two example workflow files were updated to fix the missing 'report' verb, with no unrelated modifications to Rust code or other assets.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch infra-306-example-yml-report-verb

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.

@ghost

ghost commented Jun 12, 2026

Copy link
Copy Markdown

Ready to review this PR? Stage has broken it down into 1 individual chapter for you:

Title
1 Add report subcommand to example workflows
Open in Stage

Chapters generated by Stage for commit 9227b44 on Jun 12, 2026 10:01pm UTC.

@github-actions

Copy link
Copy Markdown
Contributor

📄 Rendered report preview

All golden examples regenerated cleanly.

🟡 Golden examples

Committed to examples/ and byte-identity gated — the canonical reports contributors and consumers browse. Stable across PRs.

Report View Download
playground-report.html ▶ Open ↗ ⬇ Download
diff-showcase-report.html ▶ Open ↗ ⬇ Download
jaffle-shop-report.html ▶ Open ↗ ⬇ Download

🐶 Live dogfood preview

This PR doesn't touch dbt-project/, so there's no live dogfood preview.

▶ Open ↗ opens the report in your browser in one click —
published to this repo's GitHub Pages under /pr-309/.
⬇ Download fetches the same self-contained HTML as a workflow
artifact (auth-gated; works fully offline). Either way the report
makes zero external resource requests.

The Pages preview may take ~1 min to update after this comment
posts. On PRs from forks the Open link is unavailable (read-only
token) — use Download.

Alternative: GitHub CLI
# gh CLI >= 2.63 extracts into ./report-preview-playground/.
gh run download 27445548743 -R breezy-bays-labs/cute-dbt -n report-preview-playground
open report-preview-playground/playground-report.html

Posted by report-preview.yml for 9227b44ccc8dfaf2f1fa4084c1adc465e4c50c60. Affordance only — never blocks merge.

@cmbays
cmbays merged commit 1d674fe into main Jun 12, 2026
33 checks passed
@cmbays
cmbays deleted the infra-306-example-yml-report-verb branch June 12, 2026 22:07
github-actions Bot added a commit that referenced this pull request Jun 12, 2026
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.

fix: examples/cute-dbt-pr-review.yml missing the report verb — exits 2 since #100

1 participant