Skip to content

🎨 Palette: 터미널 좜λ ₯ 및 μ—λŸ¬ λ©”μ‹œμ§€ UX/DX κ°œμ„  - #333

Merged
seonghobae merged 1 commit into
developfrom
palette-cli-ux-improvements-13316103699188961265
Jul 23, 2026
Merged

🎨 Palette: 터미널 좜λ ₯ 및 μ—λŸ¬ λ©”μ‹œμ§€ UX/DX κ°œμ„ #333
seonghobae merged 1 commit into
developfrom
palette-cli-ux-improvements-13316103699188961265

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

πŸ’‘ What: CLI 터미널 좜λ ₯μ—μ„œ μ—λŸ¬μ™€ 도움말 λ©”μ‹œμ§€μ˜ 접두어λ₯Ό λͺ…ν™•ν•˜κ²Œ ν‘œμ€€ν™”ν•˜κ³ (❌ Error:, πŸ’‘ Hint:), 리포트 μˆ˜λŸ‰ 좜λ ₯ μ‹œ ν•˜λ“œμ½”λ”©λœ λ³΅μˆ˜ν˜• λŒ€μ‹  λ‹¨μˆ˜/볡수λ₯Ό λ¬Έλ§₯에 맞게 λ™μ μœΌλ‘œ ν‘œμ‹œν•˜λ„λ‘ κ°œμ„ ν–ˆμŠ΅λ‹ˆλ‹€.

🎯 Why:

  • μΌκ΄€λ˜μ§€ μ•Šμ€ μ—λŸ¬ λ©”μ‹œμ§€μ™€ 힌트 ꡬ문은 μ‚¬μš©μžκ°€ 문제λ₯Ό λΉ λ₯΄κ²Œ νŒŒμ•…ν•˜κ³  μ‘°μΉ˜ν•˜λŠ” 데 λ°©ν•΄κ°€ λ©λ‹ˆλ‹€. λͺ…ν™•ν•œ 접두어λ₯Ό μΆ”κ°€ν•˜μ—¬ 가독성과 μ‚¬μš©μž κ²½ν—˜(DX)을 λ†’μ˜€μŠ΅λ‹ˆλ‹€.
  • 1 components와 같은 μ–΄μƒ‰ν•œ 문법적 였λ₯˜λ₯Ό λ°©μ§€ν•˜μ—¬ 개발자 λ„κ΅¬λ‘œμ„œμ˜ ν’ˆμ§ˆκ³Ό 신뒰도λ₯Ό ν–₯μƒμ‹œν‚€κΈ° μœ„ν•¨μž…λ‹ˆλ‹€.

πŸ“Έ Before/After:

  • Before: ❌ Cannot start dashboard on 127.0.0.1:8787 (Address already in use).
  • After: ❌ Error: Cannot start dashboard on 127.0.0.1:8787 (Address already in use).
  • Before: πŸ’‘ Pass a free port with --port.
  • After: πŸ’‘ Hint: Pass a free port with --port.
  • Before: πŸ“¦ SBOM (1 components) written: ...
  • After: πŸ“¦ SBOM (1 component) written: ...

β™Ώ Accessibility: 슀크린 리더 μ‚¬μš©μžλ₯Ό λΉ„λ‘―ν•œ λͺ¨λ“  μ‚¬μš©μžκ°€ 'μ—λŸ¬'와 '힌트'λ₯Ό λͺ…μ‹œμ μΈ ν…μŠ€νŠΈ(Error:, Hint:)λ₯Ό 톡해 이λͺ¨μ§€μ™€ 관계없이 λ™μΌν•˜κ²Œ 인식할 수 μžˆλ„λ‘ 정보 접근성을 κ°œμ„ ν–ˆμŠ΅λ‹ˆλ‹€.


PR created automatically by Jules for task 13316103699188961265 started by @seonghobae

@google-labs-jules

Copy link
Copy Markdown

πŸ‘‹ Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a πŸ‘€ emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

@github-project-automation github-project-automation Bot moved this to Backlog in Security Jul 18, 2026
Copilot AI review requested due to automatic review settings July 23, 2026 01:53

Copilot AI 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.

Pull request overview

This PR improves the AppGuardrail CLI’s terminal UX/accessibility by standardizing error/hint message prefixes (❌ Error:, πŸ’‘ Hint:) and fixing awkward pluralization in SBOM output.

Changes:

  • Standardize CLI error and hint prefixes across scan/report/serve/dashboard/hook outputs.
  • Add singular/plural handling for SBOM component counts (e.g., 1 component vs 2 components).
  • Update unit tests to reflect the new output strings and minor formatting cleanups.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
uv.lock Adds a uv lock file for the project (editable local package).
tests/test_appguardrail.py Updates assertions for the new Error: / Hint: prefixed output; minor formatting cleanup.
scanner/cli/appguardrail.py Implements standardized CLI prefixes and SBOM pluralization logic; includes some formatting-only changes.
.jules/palette.md Documents the learned convention for standardized CLI prefixes and dynamic pluralization.

Comment thread scanner/cli/appguardrail.py
Copilot AI review requested due to automatic review settings July 23, 2026 02:05
@seonghobae
seonghobae enabled auto-merge (squash) July 23, 2026 02:05

Copilot AI 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.

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.

Comments suppressed due to low confidence (2)

scanner/cli/appguardrail.py:64

  • Avoid using a backslash line continuation for imports; it’s inconsistent with the surrounding parenthesized-import style and is easy to break during edits. A single-line import fits here and is clearer.
from appguardrail_core.findings import \
    is_deploy_blocking as core_is_deploy_blocking

scanner/cli/appguardrail.py:3046

  • Same redundancy as above: _console_print() already calls _format_msg() on strings, so the explicit _format_msg() wrapper is unnecessary and may lead to double-formatting in the future.
        _console_print(
            _format_msg(
                f"\nπŸ’‘ Hint: Run 'appguardrail review' to get an AI prompt for fixing {these_word}."
            )
        )

Comment thread scanner/cli/appguardrail.py
Comment thread scanner/cli/appguardrail.py
- μ—λŸ¬ λ©”μ‹œμ§€μ— `❌ Error:` 접두어 μΆ”κ°€
- 팁 λ©”μ‹œμ§€μ— `πŸ’‘ Hint:` 접두어 μΆ”κ°€
- `components` 좜λ ₯ μ‹œ λ‹¨μˆ˜/볡수 동적 처리 μΆ”κ°€
- `.jules/palette.md` 파일 μ—…λ°μ΄νŠΈ
- κ΄€λ ¨ ν…ŒμŠ€νŠΈ μ½”λ“œ 100% λŒ€μ‘ 및 톡과
Copilot AI review requested due to automatic review settings July 23, 2026 02:35
@seonghobae
seonghobae force-pushed the palette-cli-ux-improvements-13316103699188961265 branch from 1059095 to 78a4d27 Compare July 23, 2026 02:35
@opencode-agent
opencode-agent Bot disabled auto-merge July 23, 2026 02:35

Copilot AI 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.

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (2)

scanner/cli/appguardrail.py:64

  • Avoid using a backslash line-continuation in an import; it’s fragile (whitespace-sensitive) and inconsistent with the surrounding parenthesized imports. Use a normal single-line import (or parentheses) instead.
from appguardrail_core.findings import \
    is_deploy_blocking as core_is_deploy_blocking

scanner/cli/appguardrail.py:3074

  • This header currently prints "Hint: Tips:", which is redundant and makes the output less clear. Prefer a single label (e.g., "Tips:") to keep the section header concise and consistent.
    _console_print("πŸ’‘ Hint: Tips:")

@seonghobae
seonghobae merged commit 3a727c0 into develop Jul 23, 2026
34 checks passed
@seonghobae
seonghobae deleted the palette-cli-ux-improvements-13316103699188961265 branch July 23, 2026 02:44
@github-project-automation github-project-automation Bot moved this from Backlog to Done in Security Jul 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants