Skip to content

fix: apply prettier formatting to review.md#1167

Merged
Hweinstock merged 1 commit into
mainfrom
fix/prettier-formatting
May 7, 2026
Merged

fix: apply prettier formatting to review.md#1167
Hweinstock merged 1 commit into
mainfrom
fix/prettier-formatting

Conversation

@Hweinstock
Copy link
Copy Markdown
Contributor

@Hweinstock Hweinstock commented May 7, 2026

Description

Fixes a prose wrapping issue in .github/harness/prompts/review.md where a line exceeded the configured printWidth: 120 setting. Prettier with proseWrap: always now wraps it correctly.

Related Issue

Closes #1164

Documentation PR

N/A

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe): Code style fix (prettier formatting)

Testing

How have you tested the change?

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the
terms of your choice.

@github-actions github-actions Bot added size/xs PR size: XS agentcore-harness-reviewing AgentCore Harness review in progress labels May 7, 2026
Copy link
Copy Markdown

@agentcore-cli-automation agentcore-cli-automation left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. The change correctly wraps the prose at the configured printWidth: 120 per proseWrap: always in .prettierrc, fixing the format:check failure from #1164. Line 17 is now 115 chars (within 120), and the line break occurs at a natural word boundary. Minimal, targeted fix — no issues.

@github-actions github-actions Bot removed the agentcore-harness-reviewing AgentCore Harness review in progress label May 7, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 7, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 43.19% 9034 / 20915
🔵 Statements 42.47% 9592 / 22585
🔵 Functions 40.02% 1558 / 3893
🔵 Branches 40% 5811 / 14527
Generated in workflow #2623 for commit 6c2de54 by the Vitest Coverage Report Action

@Hweinstock Hweinstock marked this pull request as ready for review May 7, 2026 18:49
@Hweinstock Hweinstock requested a review from a team May 7, 2026 18:49
@Hweinstock Hweinstock merged commit ab82c66 into main May 7, 2026
27 checks passed
@Hweinstock Hweinstock deleted the fix/prettier-formatting branch May 7, 2026 19:03
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.

fail: mainline failing with format failure

3 participants