Skip to content

Re-organize harness documentation in README#1656

Open
vivdalal wants to merge 1 commit into
mainfrom
readme-updates
Open

Re-organize harness documentation in README#1656
vivdalal wants to merge 1 commit into
mainfrom
readme-updates

Conversation

@vivdalal

Copy link
Copy Markdown

Expanded the harness section in README to clarify its purpose and usage.

Description

Related Issue

Closes #

Documentation PR

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

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.

Expanded the harness section in README to clarify its purpose and usage.
@vivdalal vivdalal requested a review from a team June 27, 2026 10:57
@github-actions github-actions Bot added the size/s PR size: S label Jun 27, 2026
@github-actions github-actions Bot added the agentcore-harness-reviewing AgentCore Harness review in progress label Jun 27, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Jun 27, 2026
@agentcore-devx-automation

Copy link
Copy Markdown
Contributor

Claude Security Review: no high-confidence findings. (run)

@agentcore-devx-automation agentcore-devx-automation Bot removed the claude-security-reviewing Claude Code /security-review in progress label Jun 27, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.21.1.tgz

How to install

gh release download pr-1656-tarball --repo aws/agentcore-cli --pattern "*.tgz" --dir /tmp/pr-tarball
npm install -g /tmp/pr-tarball/aws-agentcore-0.21.1.tgz

@agentcore-cli-automation agentcore-cli-automation 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.

Docs-only reordering — moves the Harness section up under Resource Management and promotes the Harness bullet to the top of Capabilities. Verified the moved content is byte-identical to what existed before, and the referenced commands (add harness, add tool, add skill, export harness) all exist in the codebase. LGTM.

Minor note (non-blocking): the PR description says "Expanded the harness section," but nothing in the section content actually changed — it's purely a relocation. Worth tweaking the description if you want it to match what landed.

@github-actions github-actions Bot removed the agentcore-harness-reviewing AgentCore Harness review in progress label Jun 27, 2026
@vivdalal vivdalal changed the title Enhance harness documentation in README Re-organize harness documentation in README Jun 27, 2026
@github-actions

Copy link
Copy Markdown
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 37.62% 13848 / 36804
🔵 Statements 36.89% 14725 / 39915
🔵 Functions 32.16% 2368 / 7363
🔵 Branches 31.57% 9212 / 29171
Generated in workflow #3881 for commit ecda548 by the Vitest Coverage Report Action

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

Labels

size/s PR size: S

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants