Skip to content

fix(results): hide Scope config row when it equals the task's default#1101

Draft
gregmagolan wants to merge 1 commit into
mainfrom
fix-hide-default-scope
Draft

fix(results): hide Scope config row when it equals the task's default#1101
gregmagolan wants to merge 1 commit into
mainfrom
fix-hide-default-scope

Conversation

@gregmagolan
Copy link
Copy Markdown
Member

The Configuration line on gazelle and format status surfaces always
rendered the Scope row — including Scope: all on gazelle (default
all) and Scope: changed on format (default changed). That's
just noise; the user only cares when they deviated from the default.

The fix threads a per-task _DEFAULT_SCOPE constant into each task's
_config_items and suppresses the row when the value equals the
default. The non-default rendered case (and the changed — N changed dirs count suffix it carries on gazelle) is unchanged.


Changes are visible to end-users: yes

  • Searched for relevant documentation and updated as needed: yes
  • Breaking change (forces users to change their own code or config): no
  • Suggested release notes appear below: yes

Suggested release notes

  • Status surfaces (GitHub Status Checks, Buildkite annotations, PR comments) no longer render the Scope: <default> Configuration row for gazelle and format. The row still renders when the user picked a non-default --scope.

Test plan

  • Covered by existing test cases
  • New test cases added
    • test-gazelle-template-snapshots: scenarios 1–12 now omit the Scope row (default all); 13–15 still render it (non-default changed).
    • test-format-template-snapshots: scenarios 1, 2, 4–9, 11–14 now omit the Scope row (default changed); 3 and 10 still render it (non-default all).

The Configuration line on gazelle and format status surfaces always
rendered the Scope row — including `Scope: all` on gazelle (default
`all`) and `Scope: changed` on format (default `changed`). That's just
noise; the user only cares when they deviated from the default. Now
the row only renders when the scope value differs from the task's
default. Non-default scope (and the `changed — N changed dirs` count
suffix it carries on gazelle) is unchanged.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@aspect-workflows
Copy link
Copy Markdown

aspect-workflows Bot commented May 19, 2026

✨ Aspect Workflows Tasks

📅 Tue May 19 11:57:10 UTC 2026

✅ 16 successful tasks

  • ✅ build (build-gha) · ⏱ 33.7s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Bazel build complete (161 built)
  • ✅ build (build-gha-debug) · ⏱ 2m 54s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Bazel build complete (161 built)
  • ✅ format (buildifier-gha) · ⏱ 19.3s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Format complete (clean)
  • ✅ format (buildifier-gha-debug) · ⏱ 24.1s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Format complete (clean)
  • ✅ delivery (delivery-gha) · ⏱ 57s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Delivery complete (1 delivered)
  • ✅ delivery (delivery-gha-debug) · ⏱ 5m 10s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Delivery complete (1 delivered)
  • ✅ format (format-gha) · ⏱ 20.3s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Format complete (clean)
  • ✅ format (format-gha-debug) · ⏱ 24.9s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Format complete (clean)
  • ✅ gazelle (gazelle-from-source-gha) · ⏱ 14.8s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Gazelle complete (clean)
  • ✅ gazelle (gazelle-from-source-gha-debug) · ⏱ 20.3s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Gazelle complete (clean)
  • ✅ gazelle (gazelle-gha) · ⏱ 16s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Gazelle complete (clean)
  • ✅ gazelle (gazelle-gha-debug) · ⏱ 17.1s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Gazelle complete (clean)
  • ✅ lint (lint-gha) · ⏱ 16.1s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Lint complete (clean)
  • ✅ lint (lint-gha-debug) · ⏱ 17.9s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Lint complete (clean)
  • ✅ test (test-gha) · ⏱ 24.1s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Bazel test complete (25/25 passed · 25 cached)
  • ✅ test (test-gha-debug) · ⏱ 1m 6s · ✨ Aspect · 🐙 GitHub Actions · ☑️ Check
    💬 Bazel test complete (24/24 passed · 24 cached)

⏱ Last updated Tue May 19 12:03:27 UTC 2026 · 📊 GitHub API quota 1,552/15,000 (10% used, resets in 52m)
🚀 Powered by Aspect CLIAspect Build · X · LinkedIn · YouTube

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.

1 participant