Skip to content

fix: shorten disabled policy generate description to prevent truncation#7

Merged
jesseturner21 merged 1 commit intomainfrom
fix/policy-generate-description-truncation
Mar 20, 2026
Merged

fix: shorten disabled policy generate description to prevent truncation#7
jesseturner21 merged 1 commit intomainfrom
fix/policy-generate-description-truncation

Conversation

@jesseturner21
Copy link
Owner

Summary

  • Shortened the disabled "Generate a Cedar policy" option description from "Requires deployed engine — run deploy first" to "Deploy engine first"
  • The previous text was too long and got truncated by the SelectList component's wrap="truncate" rendering, making the end of the description invisible in narrow terminals

Evidence

Before: The description "Requires deployed engine — run deploy first" gets cut off on terminals narrower than ~80 columns due to wrap="truncate" in the SelectList component.

After: All three source method options render fully:

❯ Select a Cedar policy file - From your project
  Write a Cedar policy - Type Cedar directly
  Generate a Cedar policy - Deploy engine first

Verified via TUI test harness — all three options display completely without truncation.

Test plan

  • Build succeeds (npm run build)
  • TUI test: navigate to Add Policy → source method step, verify all descriptions visible
  • Lint/prettier/typecheck pass (pre-commit hooks)

🤖 Generated with Claude Code

The "Generate a Cedar policy" option's disabled description was too long
("Requires deployed engine — run `deploy` first") and got cut off in
narrow terminals. Shortened to "Deploy engine first".

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@github-actions github-actions bot added the size/xs PR size: XS label Mar 20, 2026
@github-actions
Copy link

Package Tarball

aws-agentcore-0.3.0-preview.6.1.tgz

How to install

npm install https://github.com/jesseturner21/agentcore-cli/releases/download/pr-7-tarball/aws-agentcore-0.3.0-preview.6.1.tgz

@github-actions
Copy link

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 41.48% 4976 / 11995
🔵 Statements 41.1% 5272 / 12825
🔵 Functions 40.3% 910 / 2258
🔵 Branches 42.9% 3219 / 7503
Generated in workflow #32 for commit 9accdc7 by the Vitest Coverage Report Action

@jesseturner21 jesseturner21 merged commit b2869e9 into main Mar 20, 2026
18 checks passed
@jesseturner21
Copy link
Owner Author

do a test by creating a project, then adding an agent deploying the
agent invoking after it is deployed adn then removing the agent and
deploying to apply the removal of the agent that it deletes, perform this
and make a pr to jesseturner21 fork of agentcore-cli with the svg in a pr
comment as evidence for completion with the flow you went through
this is a prompt i am testing
putting here. to save

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xs PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant