Skip to content

feat: add archive command for batch evaluations and recommendations - #1112

Merged
padmak30 merged 3 commits into
mainfrom
feat/archive
May 4, 2026
Merged

feat: add archive command for batch evaluations and recommendations#1112
padmak30 merged 3 commits into
mainfrom
feat/archive

Conversation

@padmak30

@padmak30padmak30 commented May 4, 2026

Copy link
Copy Markdown
Contributor

Description

  • Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
    history file.

    • Extracts a shared getRegion() helper into src/cli/commands/shared/region-utils.ts, removing duplicate implementations from pause/command.tsx and stop/command.tsx.
    • Adds src/cli/operations/archive/archive-storage.ts with deleteLocalBatchEvalRun and deleteLocalRecommendationRun, locating per-ID JSON files in agentcore/.cli/batch-eval-results/ and
      agentcore/.cli/recommendations/ and deleting them. Exports BATCH_EVAL_RESULTS_DIR and RECOMMENDATIONS_DIR from their existing storage modules to enable this.
    • Registers archive in CLI_ONLY_COMMANDS in TUI utils — consistent with stop, pause, resume.

  • Usage

    • Archive a batch evaluation (deletes from service and clears local history)
      agentcore archive batch-evaluation --id

    • Archive a recommendation
      agentcore archive recommendation --id

    • With explicit region
      agentcore archive batch-evaluation --id --region us-west-2

    • JSON output for scripting
      agentcore archive batch-evaluation --id --json
      agentcore archive recommendation --id --json

Type of Change

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

Testing

How have you tested the change? Added unit tests and e2e tests

  • 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

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

@padmak30
padmak30 requested a review from a teamMay 4, 2026 19:14
@github-actionsgithub-actionsBot added size/xl PR size: XL agentcore-harness-reviewing AgentCore Harness review in progress and removed agentcore-harness-reviewing AgentCore Harness review in progress labels May 4, 2026
@github-actions

github-actionsBot commented May 4, 2026•
edited
Loading

Copy link
Copy Markdown
Contributor

Coverage Report

StatusCategoryPercentageCovered / Total
🔵Lines43.04%8989 / 20883
🔵Statements42.32%9544 / 22550
🔵Functions39.86%1550 / 3888
🔵Branches39.92%5794 / 14511
Generated in workflow #2374 for commit ccb556c by the Vitest Coverage Report Action

@agentcore-cli-automation

agentcore-cli-automation commented May 4, 2026

Copy link
Copy Markdown

The new archive subcommands call DeleteBatchEvaluation and DeleteRecommendation, but docs/policies/iam-policy-user.json (the BatchEvalAndRecommendation statement that was just synced in #1092) doesn't include those IAM actions. Users following the documented IAM policy won't be able to run archive.

Please add bedrock-agentcore:DeleteBatchEvaluation and bedrock-agentcore:DeleteRecommendation to that statement. (Same gap exists today for stop batch-evaluation / StopBatchEvaluation — might be worth fixing in the same commit since #1092 just reconciled this policy.)

@github-actionsgithub-actionsBot added size/xl PR size: XL and removed size/xl PR size: XL labels May 4, 2026
notgitika
notgitika previously approved these changes May 4, 2026
View reviewed changes

@notgitikanotgitika left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM thank you!

Wonder if we can see any use case where this can be extended to some other resources in the CLI. Curious your thoughts!

Comment threadsrc/cli/operations/archive/archive-storage.ts
@github-actionsgithub-actionsBot added size/xl PR size: XL and removed size/xl PR size: XL labels May 4, 2026
@notgitika
notgitika self-requested a review May 4, 2026 20:49
notgitika
notgitika approved these changes May 4, 2026
View reviewed changes

@notgitikanotgitika left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@padmak30
padmak30 merged commit 7586092 into mainMay 4, 2026
25 checks passed
@padmak30
padmak30 deleted the feat/archive branch May 4, 2026 20:52
padmak30 added a commit that referenced this pull request May 5, 2026
@padmak30
feat: add archive command for batch evaluations and recommendations (#…
0abe589
…1112) (#1121)
* feat: add archive command for batch evaluations and recommendations.
Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
history file.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@notgitikanotgitikanotgitika approved these changes

Assignees

No one assigned

Labels

size/xlPR size: XL

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@padmak30@agentcore-cli-automation@notgitika
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all 
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
feat: add archive command for batch evaluations and recommendations by padmak30 · Pull Request #1112 · aws/agentcore-cli · GitHub
Skip to content
feat: add archive command for batch evaluations and recommendations - #1112
Merged
padmak30 merged 3 commits into
mainfrom 
feat/archive
May 4, 2026
Merged
feat: add archive command for batch evaluations and recommendations#1112
padmak30 merged 3 commits into
mainfrom 
feat/archive
Conversation
@padmak30
@padmak30padmak30
commented
May 4, 2026

Copy link


Copy Markdown

Contributor
Description
  • Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
    
history file.
    • Extracts a shared getRegion() helper into src/cli/commands/shared/region-utils.ts, removing duplicate implementations from pause/command.tsx and stop/command.tsx.
    • Adds src/cli/operations/archive/archive-storage.ts with deleteLocalBatchEvalRun and deleteLocalRecommendationRun, locating per-ID JSON files in agentcore/.cli/batch-eval-results/ and
      
agentcore/.cli/recommendations/ and deleting them. Exports BATCH_EVAL_RESULTS_DIR and RECOMMENDATIONS_DIR from their existing storage modules to enable this.
    • Registers archive in CLI_ONLY_COMMANDS in TUI utils — consistent with stop, pause, resume.
  • Usage
    • Archive a batch evaluation (deletes from service and clears local history)
      
agentcore archive batch-evaluation --id 
    • Archive a recommendation
      
agentcore archive recommendation --id 
    • With explicit region
      
agentcore archive batch-evaluation --id --region us-west-2
    • JSON output for scripting
      
agentcore archive batch-evaluation --id --json
      
agentcore archive recommendation --id --json
Type of Change
  •  Bug fix
  •  New feature
  •  Breaking change
  •  Documentation update
  •  Other (please describe):
Testing
How have you tested the change? Added unit tests and e2e tests
  •  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
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the

terms of your choice.
@padmak30
padmak30
requested a review
from a teamMay 4, 2026 19:14
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
agentcore-harness-reviewing
AgentCore Harness review in progress
and removed

agentcore-harness-reviewing
AgentCore Harness review in progress
labels
May 4, 2026
@github-actions
github-actionsBot
commented
May 4, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Coverage Report
StatusCategoryPercentageCovered / Total
🔵Lines43.04%8989 / 20883
🔵Statements42.32%9544 / 22550
🔵Functions39.86%1550 / 3888
🔵Branches39.92%5794 / 14511
Generated in workflow #2374 for commit ccb556c by the Vitest Coverage Report Action
@agentcore-cli-automation
agentcore-cli-automation
commented
May 4, 2026

Copy link


Copy Markdown

The new archive subcommands call DeleteBatchEvaluation and DeleteRecommendation, but docs/policies/iam-policy-user.json (the BatchEvalAndRecommendation statement that was just synced in #1092) doesn't include those IAM actions. Users following the documented IAM policy won't be able to run archive.
Please add bedrock-agentcore:DeleteBatchEvaluation and bedrock-agentcore:DeleteRecommendation to that statement. (Same gap exists today for stop batch-evaluation / StopBatchEvaluation — might be worth fixing in the same commit since #1092 just reconciled this policy.)
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
and removed

size/xl
PR size: XL
labels
May 4, 2026
notgitika
notgitika
previously approved these changes
May 4, 2026
View reviewed changes
@notgitikanotgitika
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

LGTM thank you!
Wonder if we can see any use case where this can be extended to some other resources in the CLI. Curious your thoughts!
Comment threadsrc/cli/operations/archive/archive-storage.ts
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
and removed

size/xl
PR size: XL
labels
May 4, 2026
@notgitika
notgitika
self-requested a review
May 4, 2026 20:49
notgitika
notgitika
approved these changes
May 4, 2026
View reviewed changes
@notgitikanotgitika
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

LGTM
@padmak30
padmak30
merged commit 7586092
into
mainMay 4, 2026

25 checks passed

@padmak30
padmak30
deleted the

feat/archive

branch
May 4, 2026 20:52
padmak30 added a commit
that referenced
this pull request
May 5, 2026
@padmak30
feat: add archive command for batch evaluations and recommendations (#…
0abe589
…1112) (#1121)
* feat: add archive command for batch evaluations and recommendations.
Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
history file.
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@notgitikanotgitikanotgitika approved these changes

Assignees


No one assigned


Labels

size/xlPR size: XL

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@padmak30@agentcore-cli-automation@notgitika

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Force GitHub README to respect dark mode
(function() {
var style = document.createElement('style');
style.textContent = '
.markdown-body {
color-scheme: dark light;
}
.markdown-body pre { background: #161b22 !important; }
.markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }
.markdown-body table th, .markdown-body table td { border-color: #30363d !important; }
.markdown-body img { background: #0d1117; }
.markdown-body blockquote { border-left-color: #8b949e; }
.markdown-body hr { border-color: #30363d; }
';
document.head.appendChild(style);
})();
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
feat: add archive command for batch evaluations and recommendations by padmak30 · Pull Request #1112 · aws/agentcore-cli · GitHub
Skip to content
feat: add archive command for batch evaluations and recommendations - #1112
Merged
padmak30 merged 3 commits into
mainfrom 
feat/archive
May 4, 2026
Merged
feat: add archive command for batch evaluations and recommendations#1112
padmak30 merged 3 commits into
mainfrom 
feat/archive
Conversation
@padmak30
@padmak30padmak30
commented
May 4, 2026

Copy link


Copy Markdown

Contributor
Description
  • Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
    
history file.
    • Extracts a shared getRegion() helper into src/cli/commands/shared/region-utils.ts, removing duplicate implementations from pause/command.tsx and stop/command.tsx.
    • Adds src/cli/operations/archive/archive-storage.ts with deleteLocalBatchEvalRun and deleteLocalRecommendationRun, locating per-ID JSON files in agentcore/.cli/batch-eval-results/ and
      
agentcore/.cli/recommendations/ and deleting them. Exports BATCH_EVAL_RESULTS_DIR and RECOMMENDATIONS_DIR from their existing storage modules to enable this.
    • Registers archive in CLI_ONLY_COMMANDS in TUI utils — consistent with stop, pause, resume.
  • Usage
    • Archive a batch evaluation (deletes from service and clears local history)
      
agentcore archive batch-evaluation --id 
    • Archive a recommendation
      
agentcore archive recommendation --id 
    • With explicit region
      
agentcore archive batch-evaluation --id --region us-west-2
    • JSON output for scripting
      
agentcore archive batch-evaluation --id --json
      
agentcore archive recommendation --id --json
Type of Change
  •  Bug fix
  •  New feature
  •  Breaking change
  •  Documentation update
  •  Other (please describe):
Testing
How have you tested the change? Added unit tests and e2e tests
  •  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
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the

terms of your choice.
@padmak30
padmak30
requested a review
from a teamMay 4, 2026 19:14
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
agentcore-harness-reviewing
AgentCore Harness review in progress
and removed

agentcore-harness-reviewing
AgentCore Harness review in progress
labels
May 4, 2026
@github-actions
github-actionsBot
commented
May 4, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Coverage Report
StatusCategoryPercentageCovered / Total
🔵Lines43.04%8989 / 20883
🔵Statements42.32%9544 / 22550
🔵Functions39.86%1550 / 3888
🔵Branches39.92%5794 / 14511
Generated in workflow #2374 for commit ccb556c by the Vitest Coverage Report Action
@agentcore-cli-automation
agentcore-cli-automation
commented
May 4, 2026

Copy link


Copy Markdown

The new archive subcommands call DeleteBatchEvaluation and DeleteRecommendation, but docs/policies/iam-policy-user.json (the BatchEvalAndRecommendation statement that was just synced in #1092) doesn't include those IAM actions. Users following the documented IAM policy won't be able to run archive.
Please add bedrock-agentcore:DeleteBatchEvaluation and bedrock-agentcore:DeleteRecommendation to that statement. (Same gap exists today for stop batch-evaluation / StopBatchEvaluation — might be worth fixing in the same commit since #1092 just reconciled this policy.)
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
and removed

size/xl
PR size: XL
labels
May 4, 2026
notgitika
notgitika
previously approved these changes
May 4, 2026
View reviewed changes
@notgitikanotgitika
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

LGTM thank you!
Wonder if we can see any use case where this can be extended to some other resources in the CLI. Curious your thoughts!
Comment threadsrc/cli/operations/archive/archive-storage.ts
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
and removed

size/xl
PR size: XL
labels
May 4, 2026
@notgitika
notgitika
self-requested a review
May 4, 2026 20:49
notgitika
notgitika
approved these changes
May 4, 2026
View reviewed changes
@notgitikanotgitika
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

LGTM
@padmak30
padmak30
merged commit 7586092
into
mainMay 4, 2026

25 checks passed

@padmak30
padmak30
deleted the

feat/archive

branch
May 4, 2026 20:52
padmak30 added a commit
that referenced
this pull request
May 5, 2026
@padmak30
feat: add archive command for batch evaluations and recommendations (#…
0abe589
…1112) (#1121)
* feat: add archive command for batch evaluations and recommendations.
Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
history file.
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@notgitikanotgitikanotgitika approved these changes

Assignees


No one assigned


Labels

size/xlPR size: XL

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@padmak30@agentcore-cli-automation@notgitika

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Highlight search terms from Google/DuckDuckGo/Bing referrer
(function() {
var ref = document.referrer;
var terms = [];
if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {
var url = new URL(ref);
var q = url.searchParams.get('q') || url.searchParams.get('p');
if (q) {
terms = q.split(/\s+/).filter(function(t) { return t.length > 2; });
}
}
if (terms.length === 0) return;
var style = document.createElement('style');
style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';
document.head.appendChild(style);
function highlight(node) {
if (node.nodeType === 3) { // text node
var text = node.textContent;
var found = false;
terms.forEach(function(term) {
var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi');
if (regex.test(text)) {
found = true;
var frag = document.createDocumentFragment();
var parts = text.split(regex);
parts.forEach(function(part, i) {
if (i % 2 === 0) {
frag.appendChild(document.createTextNode(part));
} else {
var span = document.createElement('span');
span.className = 'userscript-highlight';
span.textContent = part;
frag.appendChild(span);
}
});
node.parentNode.replaceChild(frag, node);
}
});
} else if (node.nodeType === 1 && node.childNodes) { // element
var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];
if (!skipTags.includes(node.tagName)) {
Array.from(node.childNodes).forEach(highlight);
}
}
}
highlight(document.body);
// Re-highlight on dynamic content
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1 || node.nodeType === 3) highlight(node);
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
feat: add archive command for batch evaluations and recommendations by padmak30 · Pull Request #1112 · aws/agentcore-cli · GitHub
Skip to content
feat: add archive command for batch evaluations and recommendations - #1112
Merged
padmak30 merged 3 commits into
mainfrom 
feat/archive
May 4, 2026
Merged
feat: add archive command for batch evaluations and recommendations#1112
padmak30 merged 3 commits into
mainfrom 
feat/archive
Conversation
@padmak30
@padmak30padmak30
commented
May 4, 2026

Copy link


Copy Markdown

Contributor
Description
  • Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
    
history file.
    • Extracts a shared getRegion() helper into src/cli/commands/shared/region-utils.ts, removing duplicate implementations from pause/command.tsx and stop/command.tsx.
    • Adds src/cli/operations/archive/archive-storage.ts with deleteLocalBatchEvalRun and deleteLocalRecommendationRun, locating per-ID JSON files in agentcore/.cli/batch-eval-results/ and
      
agentcore/.cli/recommendations/ and deleting them. Exports BATCH_EVAL_RESULTS_DIR and RECOMMENDATIONS_DIR from their existing storage modules to enable this.
    • Registers archive in CLI_ONLY_COMMANDS in TUI utils — consistent with stop, pause, resume.
  • Usage
    • Archive a batch evaluation (deletes from service and clears local history)
      
agentcore archive batch-evaluation --id 
    • Archive a recommendation
      
agentcore archive recommendation --id 
    • With explicit region
      
agentcore archive batch-evaluation --id --region us-west-2
    • JSON output for scripting
      
agentcore archive batch-evaluation --id --json
      
agentcore archive recommendation --id --json
Type of Change
  •  Bug fix
  •  New feature
  •  Breaking change
  •  Documentation update
  •  Other (please describe):
Testing
How have you tested the change? Added unit tests and e2e tests
  •  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
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the

terms of your choice.
@padmak30
padmak30
requested a review
from a teamMay 4, 2026 19:14
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
agentcore-harness-reviewing
AgentCore Harness review in progress
and removed

agentcore-harness-reviewing
AgentCore Harness review in progress
labels
May 4, 2026
@github-actions
github-actionsBot
commented
May 4, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Coverage Report
StatusCategoryPercentageCovered / Total
🔵Lines43.04%8989 / 20883
🔵Statements42.32%9544 / 22550
🔵Functions39.86%1550 / 3888
🔵Branches39.92%5794 / 14511
Generated in workflow #2374 for commit ccb556c by the Vitest Coverage Report Action
@agentcore-cli-automation
agentcore-cli-automation
commented
May 4, 2026

Copy link


Copy Markdown

The new archive subcommands call DeleteBatchEvaluation and DeleteRecommendation, but docs/policies/iam-policy-user.json (the BatchEvalAndRecommendation statement that was just synced in #1092) doesn't include those IAM actions. Users following the documented IAM policy won't be able to run archive.
Please add bedrock-agentcore:DeleteBatchEvaluation and bedrock-agentcore:DeleteRecommendation to that statement. (Same gap exists today for stop batch-evaluation / StopBatchEvaluation — might be worth fixing in the same commit since #1092 just reconciled this policy.)
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
and removed

size/xl
PR size: XL
labels
May 4, 2026
notgitika
notgitika
previously approved these changes
May 4, 2026
View reviewed changes
@notgitikanotgitika
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

LGTM thank you!
Wonder if we can see any use case where this can be extended to some other resources in the CLI. Curious your thoughts!
Comment threadsrc/cli/operations/archive/archive-storage.ts
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
and removed

size/xl
PR size: XL
labels
May 4, 2026
@notgitika
notgitika
self-requested a review
May 4, 2026 20:49
notgitika
notgitika
approved these changes
May 4, 2026
View reviewed changes
@notgitikanotgitika
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

LGTM
@padmak30
padmak30
merged commit 7586092
into
mainMay 4, 2026

25 checks passed

@padmak30
padmak30
deleted the

feat/archive

branch
May 4, 2026 20:52
padmak30 added a commit
that referenced
this pull request
May 5, 2026
@padmak30
feat: add archive command for batch evaluations and recommendations (#…
0abe589
…1112) (#1121)
* feat: add archive command for batch evaluations and recommendations.
Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
history file.
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@notgitikanotgitikanotgitika approved these changes

Assignees


No one assigned


Labels

size/xlPR size: XL

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@padmak30@agentcore-cli-automation@notgitika

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Strip utm_, fbclid, gclid, etc. from all links on page
(function() {
var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',
'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',
'ref', 'ref_src', 'source', 'medium', 'campaign'];
function cleanUrl(url) {
try {
var u = new URL(url, window.location.origin);
var changed = false;
trackingParams.forEach(function(p) {
if (u.searchParams.has(p)) {
u.searchParams.delete(p);
changed = true;
}
});
return changed ? u.toString() : url;
} catch (e) {
return url;
}
}
function cleanLinks() {
document.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
cleanLinks();
var observer = new MutationObserver(function(mutations) {
mutations.forEach(function(m) {
m.addedNodes.forEach(function(node) {
if (node.nodeType === 1) {
if (node.tagName === 'A') cleanLinks();
node.querySelectorAll('a[href]').forEach(function(a) {
var clean = cleanUrl(a.href);
if (clean !== a.href) a.href = clean;
});
}
});
});
});
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '
feat: add archive command for batch evaluations and recommendations by padmak30 · Pull Request #1112 · aws/agentcore-cli · GitHub
Skip to content
feat: add archive command for batch evaluations and recommendations - #1112
Merged
padmak30 merged 3 commits into
mainfrom 
feat/archive
May 4, 2026
Merged
feat: add archive command for batch evaluations and recommendations#1112
padmak30 merged 3 commits into
mainfrom 
feat/archive
Conversation
@padmak30
@padmak30padmak30
commented
May 4, 2026

Copy link


Copy Markdown

Contributor
Description
  • Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
    
history file.
    • Extracts a shared getRegion() helper into src/cli/commands/shared/region-utils.ts, removing duplicate implementations from pause/command.tsx and stop/command.tsx.
    • Adds src/cli/operations/archive/archive-storage.ts with deleteLocalBatchEvalRun and deleteLocalRecommendationRun, locating per-ID JSON files in agentcore/.cli/batch-eval-results/ and
      
agentcore/.cli/recommendations/ and deleting them. Exports BATCH_EVAL_RESULTS_DIR and RECOMMENDATIONS_DIR from their existing storage modules to enable this.
    • Registers archive in CLI_ONLY_COMMANDS in TUI utils — consistent with stop, pause, resume.
  • Usage
    • Archive a batch evaluation (deletes from service and clears local history)
      
agentcore archive batch-evaluation --id 
    • Archive a recommendation
      
agentcore archive recommendation --id 
    • With explicit region
      
agentcore archive batch-evaluation --id --region us-west-2
    • JSON output for scripting
      
agentcore archive batch-evaluation --id --json
      
agentcore archive recommendation --id --json
Type of Change
  •  Bug fix
  •  New feature
  •  Breaking change
  •  Documentation update
  •  Other (please describe):
Testing
How have you tested the change? Added unit tests and e2e tests
  •  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
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the

terms of your choice.
@padmak30
padmak30
requested a review
from a teamMay 4, 2026 19:14
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
agentcore-harness-reviewing
AgentCore Harness review in progress
and removed

agentcore-harness-reviewing
AgentCore Harness review in progress
labels
May 4, 2026
@github-actions
github-actionsBot
commented
May 4, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Coverage Report
StatusCategoryPercentageCovered / Total
🔵Lines43.04%8989 / 20883
🔵Statements42.32%9544 / 22550
🔵Functions39.86%1550 / 3888
🔵Branches39.92%5794 / 14511
Generated in workflow #2374 for commit ccb556c by the Vitest Coverage Report Action
@agentcore-cli-automation
agentcore-cli-automation
commented
May 4, 2026

Copy link


Copy Markdown

The new archive subcommands call DeleteBatchEvaluation and DeleteRecommendation, but docs/policies/iam-policy-user.json (the BatchEvalAndRecommendation statement that was just synced in #1092) doesn't include those IAM actions. Users following the documented IAM policy won't be able to run archive.
Please add bedrock-agentcore:DeleteBatchEvaluation and bedrock-agentcore:DeleteRecommendation to that statement. (Same gap exists today for stop batch-evaluation / StopBatchEvaluation — might be worth fixing in the same commit since #1092 just reconciled this policy.)
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
and removed

size/xl
PR size: XL
labels
May 4, 2026
notgitika
notgitika
previously approved these changes
May 4, 2026
View reviewed changes
@notgitikanotgitika
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

LGTM thank you!
Wonder if we can see any use case where this can be extended to some other resources in the CLI. Curious your thoughts!
Comment threadsrc/cli/operations/archive/archive-storage.ts
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
and removed

size/xl
PR size: XL
labels
May 4, 2026
@notgitika
notgitika
self-requested a review
May 4, 2026 20:49
notgitika
notgitika
approved these changes
May 4, 2026
View reviewed changes
@notgitikanotgitika
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

LGTM
@padmak30
padmak30
merged commit 7586092
into
mainMay 4, 2026

25 checks passed

@padmak30
padmak30
deleted the

feat/archive

branch
May 4, 2026 20:52
padmak30 added a commit
that referenced
this pull request
May 5, 2026
@padmak30
feat: add archive command for batch evaluations and recommendations (#…
0abe589
…1112) (#1121)
* feat: add archive command for batch evaluations and recommendations.
Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
history file.
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@notgitikanotgitikanotgitika approved these changes

Assignees


No one assigned


Labels

size/xlPR size: XL

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@padmak30@agentcore-cli-automation@notgitika

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Auto-enable theater mode on YouTube
(function() {
function tryTheater() {
var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]');
if (btn && !btn.classList.contains('activated')) {
btn.click();
}
}
// Try immediately
tryTheater();
// Try after navigation (SPA)
var lastUrl = location.href;
setInterval(function() {
if (location.href !== lastUrl) {
lastUrl = location.href;
setTimeout(tryTheater, 500);
}
}, 1000);
// Also try on player load
var observer = new MutationObserver(tryTheater);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
feat: add archive command for batch evaluations and recommendations by padmak30 · Pull Request #1112 · aws/agentcore-cli · GitHub
Skip to content
feat: add archive command for batch evaluations and recommendations - #1112
Merged
padmak30 merged 3 commits into
mainfrom 
feat/archive
May 4, 2026
Merged
feat: add archive command for batch evaluations and recommendations#1112
padmak30 merged 3 commits into
mainfrom 
feat/archive
Conversation
@padmak30
@padmak30padmak30
commented
May 4, 2026

Copy link


Copy Markdown

Contributor
Description
  • Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
    
history file.
    • Extracts a shared getRegion() helper into src/cli/commands/shared/region-utils.ts, removing duplicate implementations from pause/command.tsx and stop/command.tsx.
    • Adds src/cli/operations/archive/archive-storage.ts with deleteLocalBatchEvalRun and deleteLocalRecommendationRun, locating per-ID JSON files in agentcore/.cli/batch-eval-results/ and
      
agentcore/.cli/recommendations/ and deleting them. Exports BATCH_EVAL_RESULTS_DIR and RECOMMENDATIONS_DIR from their existing storage modules to enable this.
    • Registers archive in CLI_ONLY_COMMANDS in TUI utils — consistent with stop, pause, resume.
  • Usage
    • Archive a batch evaluation (deletes from service and clears local history)
      
agentcore archive batch-evaluation --id 
    • Archive a recommendation
      
agentcore archive recommendation --id 
    • With explicit region
      
agentcore archive batch-evaluation --id --region us-west-2
    • JSON output for scripting
      
agentcore archive batch-evaluation --id --json
      
agentcore archive recommendation --id --json
Type of Change
  •  Bug fix
  •  New feature
  •  Breaking change
  •  Documentation update
  •  Other (please describe):
Testing
How have you tested the change? Added unit tests and e2e tests
  •  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
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the

terms of your choice.
@padmak30
padmak30
requested a review
from a teamMay 4, 2026 19:14
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
agentcore-harness-reviewing
AgentCore Harness review in progress
and removed

agentcore-harness-reviewing
AgentCore Harness review in progress
labels
May 4, 2026
@github-actions
github-actionsBot
commented
May 4, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Coverage Report
StatusCategoryPercentageCovered / Total
🔵Lines43.04%8989 / 20883
🔵Statements42.32%9544 / 22550
🔵Functions39.86%1550 / 3888
🔵Branches39.92%5794 / 14511
Generated in workflow #2374 for commit ccb556c by the Vitest Coverage Report Action
@agentcore-cli-automation
agentcore-cli-automation
commented
May 4, 2026

Copy link


Copy Markdown

The new archive subcommands call DeleteBatchEvaluation and DeleteRecommendation, but docs/policies/iam-policy-user.json (the BatchEvalAndRecommendation statement that was just synced in #1092) doesn't include those IAM actions. Users following the documented IAM policy won't be able to run archive.
Please add bedrock-agentcore:DeleteBatchEvaluation and bedrock-agentcore:DeleteRecommendation to that statement. (Same gap exists today for stop batch-evaluation / StopBatchEvaluation — might be worth fixing in the same commit since #1092 just reconciled this policy.)
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
and removed

size/xl
PR size: XL
labels
May 4, 2026
notgitika
notgitika
previously approved these changes
May 4, 2026
View reviewed changes
@notgitikanotgitika
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

LGTM thank you!
Wonder if we can see any use case where this can be extended to some other resources in the CLI. Curious your thoughts!
Comment threadsrc/cli/operations/archive/archive-storage.ts
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
and removed

size/xl
PR size: XL
labels
May 4, 2026
@notgitika
notgitika
self-requested a review
May 4, 2026 20:49
notgitika
notgitika
approved these changes
May 4, 2026
View reviewed changes
@notgitikanotgitika
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

LGTM
@padmak30
padmak30
merged commit 7586092
into
mainMay 4, 2026

25 checks passed

@padmak30
padmak30
deleted the

feat/archive

branch
May 4, 2026 20:52
padmak30 added a commit
that referenced
this pull request
May 5, 2026
@padmak30
feat: add archive command for batch evaluations and recommendations (#…
0abe589
…1112) (#1121)
* feat: add archive command for batch evaluations and recommendations.
Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
history file.
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@notgitikanotgitikanotgitika approved these changes

Assignees


No one assigned


Labels

size/xlPR size: XL

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@padmak30@agentcore-cli-automation@notgitika

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Remove or un-stick sticky/fixed headers that block content
(function() {
function unstick() {
document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) {
if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {
el.style.position = 'static';
el.style.top = 'auto';
el.style.zIndex = 'auto';
}
});
}
unstick();
var observer = new MutationObserver(unstick);
observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });
})();
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '
feat: add archive command for batch evaluations and recommendations by padmak30 · Pull Request #1112 · aws/agentcore-cli · GitHub
Skip to content
feat: add archive command for batch evaluations and recommendations - #1112
Merged
padmak30 merged 3 commits into
mainfrom 
feat/archive
May 4, 2026
Merged
feat: add archive command for batch evaluations and recommendations#1112
padmak30 merged 3 commits into
mainfrom 
feat/archive
Conversation
@padmak30
@padmak30padmak30
commented
May 4, 2026

Copy link


Copy Markdown

Contributor
Description
  • Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
    
history file.
    • Extracts a shared getRegion() helper into src/cli/commands/shared/region-utils.ts, removing duplicate implementations from pause/command.tsx and stop/command.tsx.
    • Adds src/cli/operations/archive/archive-storage.ts with deleteLocalBatchEvalRun and deleteLocalRecommendationRun, locating per-ID JSON files in agentcore/.cli/batch-eval-results/ and
      
agentcore/.cli/recommendations/ and deleting them. Exports BATCH_EVAL_RESULTS_DIR and RECOMMENDATIONS_DIR from their existing storage modules to enable this.
    • Registers archive in CLI_ONLY_COMMANDS in TUI utils — consistent with stop, pause, resume.
  • Usage
    • Archive a batch evaluation (deletes from service and clears local history)
      
agentcore archive batch-evaluation --id 
    • Archive a recommendation
      
agentcore archive recommendation --id 
    • With explicit region
      
agentcore archive batch-evaluation --id --region us-west-2
    • JSON output for scripting
      
agentcore archive batch-evaluation --id --json
      
agentcore archive recommendation --id --json
Type of Change
  •  Bug fix
  •  New feature
  •  Breaking change
  •  Documentation update
  •  Other (please describe):
Testing
How have you tested the change? Added unit tests and e2e tests
  •  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
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the

terms of your choice.
@padmak30
padmak30
requested a review
from a teamMay 4, 2026 19:14
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
agentcore-harness-reviewing
AgentCore Harness review in progress
and removed

agentcore-harness-reviewing
AgentCore Harness review in progress
labels
May 4, 2026
@github-actions
github-actionsBot
commented
May 4, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Coverage Report
StatusCategoryPercentageCovered / Total
🔵Lines43.04%8989 / 20883
🔵Statements42.32%9544 / 22550
🔵Functions39.86%1550 / 3888
🔵Branches39.92%5794 / 14511
Generated in workflow #2374 for commit ccb556c by the Vitest Coverage Report Action
@agentcore-cli-automation
agentcore-cli-automation
commented
May 4, 2026

Copy link


Copy Markdown

The new archive subcommands call DeleteBatchEvaluation and DeleteRecommendation, but docs/policies/iam-policy-user.json (the BatchEvalAndRecommendation statement that was just synced in #1092) doesn't include those IAM actions. Users following the documented IAM policy won't be able to run archive.
Please add bedrock-agentcore:DeleteBatchEvaluation and bedrock-agentcore:DeleteRecommendation to that statement. (Same gap exists today for stop batch-evaluation / StopBatchEvaluation — might be worth fixing in the same commit since #1092 just reconciled this policy.)
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
and removed

size/xl
PR size: XL
labels
May 4, 2026
notgitika
notgitika
previously approved these changes
May 4, 2026
View reviewed changes
@notgitikanotgitika
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

LGTM thank you!
Wonder if we can see any use case where this can be extended to some other resources in the CLI. Curious your thoughts!
Comment threadsrc/cli/operations/archive/archive-storage.ts
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
and removed

size/xl
PR size: XL
labels
May 4, 2026
@notgitika
notgitika
self-requested a review
May 4, 2026 20:49
notgitika
notgitika
approved these changes
May 4, 2026
View reviewed changes
@notgitikanotgitika
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

LGTM
@padmak30
padmak30
merged commit 7586092
into
mainMay 4, 2026

25 checks passed

@padmak30
padmak30
deleted the

feat/archive

branch
May 4, 2026 20:52
padmak30 added a commit
that referenced
this pull request
May 5, 2026
@padmak30
feat: add archive command for batch evaluations and recommendations (#…
0abe589
…1112) (#1121)
* feat: add archive command for batch evaluations and recommendations.
Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
history file.
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@notgitikanotgitikanotgitika approved these changes

Assignees


No one assigned


Labels

size/xlPR size: XL

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@padmak30@agentcore-cli-automation@notgitika

, 'i');
if (__m === '*' || __re.test(location.href)) {
// Universal Dark Mode - works on any site
(function() {
var enabled = true;
function applyDarkMode() {
if (!enabled) return;
// Create style element if it doesn't exist
var style = document.getElementById('universal-dark-mode-style');
if (!style) {
style = document.createElement('style');
style.id = 'universal-dark-mode-style';
document.head.appendChild(style);
}
// Dark mode CSS - inverts colors but preserves images/video
style.textContent = '
/* Invert everything except media */
html {
filter: invert(1) hue-rotate(180deg) !important;
background: #1a1a2e !important;
}
/* Restore images, videos, iframes, canvas */
img, video, iframe, canvas, svg, picture, [style*="background-image"] {
filter: invert(1) hue-rotate(180deg) !important;
}
/* Preserve specific elements that should not be inverted */
.no-dark-mode, .no-dark-mode *,
[data-theme="light"], [data-theme="light"],
.ace_editor, .ace_editor *,
.CodeMirror, .CodeMirror *,
.monaco-editor, .monaco-editor *,
.markdown-body pre, .markdown-body pre *,
.highlight, .highlight *,
pre code, pre code * {
filter: none !important;
}
/* Fix common UI elements */
.modal, .popup, .dropdown-menu, .tooltip, .popover {
filter: invert(1) hue-rotate(180deg) !important;
background: #2d2d44 !important;
border-color: #444 !important;
}
/* Scrollbars */
::-webkit-scrollbar { background: #1a1a2e !important; }
::-webkit-scrollbar-thumb { background: #444 !important; }
::-webkit-scrollbar-thumb:hover { background: #555 !important; }
/* Selection */
::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }
';
}
function removeDarkMode() {
var style = document.getElementById('universal-dark-mode-style');
if (style) style.remove();
}
// Toggle with Alt+Shift+D
document.addEventListener('keydown', function(e) {
if (e.altKey && e.shiftKey && e.key === 'D') {
e.preventDefault();
enabled = !enabled;
if (enabled) {
applyDarkMode();
console.log('[Universal Dark Mode] Enabled');
} else {
removeDarkMode();
console.log('[Universal Dark Mode] Disabled');
}
}
});
// Apply on load
applyDarkMode();
// Re-apply on dynamic content
var observer = new MutationObserver(function(mutations) {
if (enabled && !document.getElementById('universal-dark-mode-style')) {
applyDarkMode();
}
});
observer.observe(document.head, { childList: true });
console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');
})();
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();
feat: add archive command for batch evaluations and recommendations by padmak30 · Pull Request #1112 · aws/agentcore-cli · GitHub
Skip to content
feat: add archive command for batch evaluations and recommendations - #1112
Merged
padmak30 merged 3 commits into
mainfrom 
feat/archive
May 4, 2026
Merged
feat: add archive command for batch evaluations and recommendations#1112
padmak30 merged 3 commits into
mainfrom 
feat/archive
Conversation
@padmak30
@padmak30padmak30
commented
May 4, 2026

Copy link


Copy Markdown

Contributor
Description
  • Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
    
history file.
    • Extracts a shared getRegion() helper into src/cli/commands/shared/region-utils.ts, removing duplicate implementations from pause/command.tsx and stop/command.tsx.
    • Adds src/cli/operations/archive/archive-storage.ts with deleteLocalBatchEvalRun and deleteLocalRecommendationRun, locating per-ID JSON files in agentcore/.cli/batch-eval-results/ and
      
agentcore/.cli/recommendations/ and deleting them. Exports BATCH_EVAL_RESULTS_DIR and RECOMMENDATIONS_DIR from their existing storage modules to enable this.
    • Registers archive in CLI_ONLY_COMMANDS in TUI utils — consistent with stop, pause, resume.
  • Usage
    • Archive a batch evaluation (deletes from service and clears local history)
      
agentcore archive batch-evaluation --id 
    • Archive a recommendation
      
agentcore archive recommendation --id 
    • With explicit region
      
agentcore archive batch-evaluation --id --region us-west-2
    • JSON output for scripting
      
agentcore archive batch-evaluation --id --json
      
agentcore archive recommendation --id --json
Type of Change
  •  Bug fix
  •  New feature
  •  Breaking change
  •  Documentation update
  •  Other (please describe):
Testing
How have you tested the change? Added unit tests and e2e tests
  •  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
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the

terms of your choice.
@padmak30
padmak30
requested a review
from a teamMay 4, 2026 19:14
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
agentcore-harness-reviewing
AgentCore Harness review in progress
and removed

agentcore-harness-reviewing
AgentCore Harness review in progress
labels
May 4, 2026
@github-actions
github-actionsBot
commented
May 4, 2026•

edited

Loading

Copy link


Copy Markdown

Contributor
Coverage Report
StatusCategoryPercentageCovered / Total
🔵Lines43.04%8989 / 20883
🔵Statements42.32%9544 / 22550
🔵Functions39.86%1550 / 3888
🔵Branches39.92%5794 / 14511
Generated in workflow #2374 for commit ccb556c by the Vitest Coverage Report Action
@agentcore-cli-automation
agentcore-cli-automation
commented
May 4, 2026

Copy link


Copy Markdown

The new archive subcommands call DeleteBatchEvaluation and DeleteRecommendation, but docs/policies/iam-policy-user.json (the BatchEvalAndRecommendation statement that was just synced in #1092) doesn't include those IAM actions. Users following the documented IAM policy won't be able to run archive.
Please add bedrock-agentcore:DeleteBatchEvaluation and bedrock-agentcore:DeleteRecommendation to that statement. (Same gap exists today for stop batch-evaluation / StopBatchEvaluation — might be worth fixing in the same commit since #1092 just reconciled this policy.)
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
and removed

size/xl
PR size: XL
labels
May 4, 2026
notgitika
notgitika
previously approved these changes
May 4, 2026
View reviewed changes
@notgitikanotgitika
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

LGTM thank you!
Wonder if we can see any use case where this can be extended to some other resources in the CLI. Curious your thoughts!
Comment threadsrc/cli/operations/archive/archive-storage.ts
@github-actionsgithub-actionsBot
added

size/xl
PR size: XL
and removed

size/xl
PR size: XL
labels
May 4, 2026
@notgitika
notgitika
self-requested a review
May 4, 2026 20:49
notgitika
notgitika
approved these changes
May 4, 2026
View reviewed changes
@notgitikanotgitika
left a comment


Copy link


Copy Markdown

Contributor
Choose a reason for hiding this comment

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

LGTM
@padmak30
padmak30
merged commit 7586092
into
mainMay 4, 2026

25 checks passed

@padmak30
padmak30
deleted the

feat/archive

branch
May 4, 2026 20:52
padmak30 added a commit
that referenced
this pull request
May 5, 2026
@padmak30
feat: add archive command for batch evaluations and recommendations (#…
0abe589
…1112) (#1121)
* feat: add archive command for batch evaluations and recommendations.
Introduces a new top-level archive command with two subcommands: archive batch-evaluation and archive recommendation. Each calls the corresponding service delete API and removes the matching local .cli/
history file.
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers

@notgitikanotgitikanotgitika approved these changes

Assignees


No one assigned


Labels

size/xlPR size: XL

Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

3 participants

@padmak30@agentcore-cli-automation@notgitika