Skip to content

feat: add telemetry notice and preference management#797

Merged
Hweinstock merged 1 commit intoaws:mainfrom
Hweinstock:feat/telemetry-phase1
Apr 9, 2026
Merged

feat: add telemetry notice and preference management#797
Hweinstock merged 1 commit intoaws:mainfrom
Hweinstock:feat/telemetry-phase1

Conversation

@Hweinstock
Copy link
Copy Markdown
Contributor

@Hweinstock Hweinstock commented Apr 9, 2026

Description

Add first-run telemetry notice and preference management commands to the CLI.

  • On first launch (no installationId in ~/.agentcore/config.json), a yellow notice informs users about upcoming anonymous usage analytics and how to opt out.
    • CLI path: notice prints before command output
    • TUI path: notice prints after the TUI exits (avoids being swallowed by the alternate screen buffer)
  • agentcore telemetry disable|enable|status subcommands for managing preferences
  • 3-tier preference resolution: env var (AGENTCORE_TELEMETRY_DISABLED) > global config > default (enabled)
  • ~/.agentcore/config.json for persistent global config (installationId, telemetry preferences)
The AgentCore CLI will soon begin collecting aggregated, anonymous usage
analytics to help improve the tool.
To opt out:          agentcore telemetry disable
To learn more:       agentcore telemetry --help

Related Issue

Closes #

Documentation PR

N/A

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

played around with packaged tarball to verify it works e2e.

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 the size/l PR size: L label Apr 9, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 9, 2026

Package Tarball

aws-agentcore-0.7.1.tgz

How to install

npm install https://github.com/aws/agentcore-cli/releases/download/pr-797-tarball/aws-agentcore-0.7.1.tgz

@Hweinstock Hweinstock force-pushed the feat/telemetry-phase1 branch from 67f27fa to 580fae2 Compare April 9, 2026 14:40
@github-actions github-actions bot added size/l PR size: L and removed size/l PR size: L labels Apr 9, 2026
@Hweinstock Hweinstock force-pushed the feat/telemetry-phase1 branch from 580fae2 to b82a818 Compare April 9, 2026 14:44
@github-actions github-actions bot added size/l PR size: L and removed size/l PR size: L labels Apr 9, 2026
@Hweinstock Hweinstock force-pushed the feat/telemetry-phase1 branch from b82a818 to c97d12d Compare April 9, 2026 14:51
@github-actions github-actions bot added size/l PR size: L and removed size/l PR size: L labels Apr 9, 2026
@Hweinstock Hweinstock force-pushed the feat/telemetry-phase1 branch from c97d12d to 7cdbba8 Compare April 9, 2026 14:54
@github-actions github-actions bot added size/l PR size: L and removed size/l PR size: L labels Apr 9, 2026
@Hweinstock Hweinstock force-pushed the feat/telemetry-phase1 branch from 7cdbba8 to a10aa17 Compare April 9, 2026 14:58
@github-actions github-actions bot added size/l PR size: L and removed size/l PR size: L labels Apr 9, 2026
@Hweinstock Hweinstock force-pushed the feat/telemetry-phase1 branch from a10aa17 to e965b09 Compare April 9, 2026 15:05
@github-actions github-actions bot added size/l PR size: L and removed size/l PR size: L labels Apr 9, 2026
@Hweinstock Hweinstock force-pushed the feat/telemetry-phase1 branch from e965b09 to 3ec7c3a Compare April 9, 2026 15:10
@github-actions github-actions bot added size/m PR size: M and removed size/l PR size: L labels Apr 9, 2026
Copy link
Copy Markdown
Contributor

@aidandaly24 aidandaly24 left a comment

Choose a reason for hiding this comment

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

Overall looks good just some comments

@Hweinstock Hweinstock force-pushed the feat/telemetry-phase1 branch from 0f537a5 to 05a592d Compare April 9, 2026 17:19
@github-actions github-actions bot removed the size/l PR size: L label Apr 9, 2026
@github-actions github-actions bot added the size/l PR size: L label Apr 9, 2026
@Hweinstock Hweinstock force-pushed the feat/telemetry-phase1 branch from 05a592d to 38d56d7 Compare April 9, 2026 17:22
@github-actions github-actions bot added size/l PR size: L and removed size/l PR size: L labels Apr 9, 2026
@Hweinstock Hweinstock force-pushed the feat/telemetry-phase1 branch from 38d56d7 to 0e8998a Compare April 9, 2026 17:39
@github-actions github-actions bot added size/l PR size: L and removed size/l PR size: L labels Apr 9, 2026
@Hweinstock Hweinstock force-pushed the feat/telemetry-phase1 branch from 0e8998a to 7517d84 Compare April 9, 2026 17:41
@github-actions github-actions bot added size/l PR size: L and removed size/l PR size: L labels Apr 9, 2026
@Hweinstock Hweinstock force-pushed the feat/telemetry-phase1 branch from 7517d84 to 6bd1445 Compare April 9, 2026 17:42
@github-actions github-actions bot added size/l PR size: L and removed size/l PR size: L labels Apr 9, 2026
@Hweinstock
Copy link
Copy Markdown
Contributor Author

lint failure from #697, will fix in a follow-up.

@Hweinstock Hweinstock merged commit fb34507 into aws:main Apr 9, 2026
20 of 21 checks passed
@Hweinstock Hweinstock deleted the feat/telemetry-phase1 branch April 9, 2026 18:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/l PR size: L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants