feat(templates): support rendering strands http template. by Hweinstock · Pull Request #2099 · aws/agentcore-cli

Hweinstock · 2026-08-25T18:25:24Z

Problem

We are not able to render real templates yet.

Solution

implement support for add runtime in core with the strands python http template as an example.

refactored addResources to modify an in-memory agentcore spec for stronger type safety and more flexibility in adding multiple resources.
refactored the template.ts file into its own directory to make it extensible to future templates.
define a TemplateRenderer interface that we can implement via handlebars.

Testing / Verification

create an agent from this template through create flow, and runtime add flow, then verify dev works for both:

bun run compile

./dist/bin/agentcore-linux-x64 project create --name testProject --template strands-python

cd testProject/

../dist/bin/agentcore-linux-x64 project add runtime --name agent2 --template strands-python

../dist/bin/agentcore-linux-x64 project dev

<seperate terminal>

 curl -X POST localhost:8080/invocations -H 'Content-type:
 application/json' -d '{ "prompt": "hello" }'

 curl -X POST localhost:8081/invocations -H 'Content-type:
 application/json' -d '{ "prompt": "hello" }'

we get responses on both

unit tests for all of this.

Future Work

This PR does not yet wire up support for memory, credentials for non-bedrock providers, or other templates. These will all be follow-ups, but their extension should be hopefully obvious based on the shape setup by the PR.

Also, we want to support overriding runtimeName on a template within the create flow (comment from previous PR that wasn't fully addressed here, but should be easy to support now with dynamic templates).

codecov-commenter · 2026-08-25T18:29:16Z

Codecov Report

❌ Patch coverage is 92.78351% with 28 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.34%. Comparing base (9ab30d3) to head (917cc91).
⚠️ Report is 4 commits behind head on refactor.

Files with missing lines	Patch %	Lines
src/handlers/project/types.ts	70.58%	10 Missing ⚠️
src/core/project/templates/renderer.ts	83.72%	7 Missing ⚠️
src/core/project/templates/runtime.ts	93.26%	7 Missing ⚠️
src/core/project/templates/harness.ts	89.74%	4 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##           refactor    #2099      +/-   ##
============================================
- Coverage     97.41%   97.34%   -0.07%     
============================================
  Files           453      456       +3     
  Lines         27637    27864     +227     
============================================
+ Hits          26922    27125     +203     
- Misses          715      739      +24

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

agentcore-devx-automation · 2026-08-25T18:30:05Z

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

agentcore-devx-automation · 2026-08-25T22:29:32Z

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

agentcore-devx-automation · 2026-08-26T01:29:32Z

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

agentcore-devx-automation · 2026-08-26T01:34:26Z

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

aidandaly24 · 2026-08-27T03:36:00Z

+}
+
+const getTemplateResolvers = (assetSource: AssetSource, templateRenderer: TemplateRenderer) => ({
+  [buildResolverKey("none", "Python")]: async (input: RuntimeResourceConfig) => {


I tried agentcore project add runtime --name mcp_agent --template hello-world-python --protocol MCP. It succeeds and writes protocol: "MCP", but the generated main.py still runs BedrockAgentCoreApp HTTP. I think the none/Python resolver should reject non-HTTP protocols the same way strands-python does.

good catch, I wasn't really testing the hello world python since i wasn't sure if we were going to remove it, but we should fix this.

aidandaly24 · 2026-08-27T03:36:01Z

+
+In a new terminal, you can invoke that server with:
+
+`agentcore project invoke --dev "What can you do"`


nit: agentcore project invoke --dev is not registered on the refactor branch. We do have local invocation through the Inspector /invocations endpoint, so should this example use dev or the Agent Inspector instead? Deployed invocation will be the top-level agentcore project invoke command once #2115 lands. I can update this README.md in that PR or those commands could be included here too.

Let me remove for now and we can add back once we know what the command will look like.

tejaskash · 2026-08-26T14:20:31Z

+In a new terminal, you can invoke that server with:
+
+`agentcore project invoke --dev "What can you do"`


Did we implement this?

removed, we can re-add once its implemented.

tejaskash · 2026-08-26T14:21:39Z

@@ -0,0 +1,714 @@
+from typing import Any


Let's discuss this today, but we've been back and forth on what the right mix is between human readable templates and flexibility to compose different features on the fly

tejaskash · 2026-08-27T13:47:41Z

+        const outputPath = join(project.rootPath, "app", input.resourceConfig.name);
+        scaffoldedPaths.push(outputPath);
+
+        const spec = await this.scaffoldRuntimeResources(outputPath, input.resourceConfig);


Could we generate uv.lock during this runtime scaffold? hello-world-python-container emits a Dockerfile that copies uv.lock and runs uv sync --frozen, but this path never creates the lockfile. A new container runtime fails on the next project dev or deploy unless the user runs uv sync first.

tejaskash · 2026-08-27T13:47:48Z

+build-backend = "hatchling.build"
+
+[project]
+name = "{{ name }}"


Could we normalize the runtime name before using it as the package name here? The CLI accepts names such as agent_, but uv rejects that value in [project].name, so uv lock and uv sync fail after the add command succeeds.

tejaskash · 2026-08-27T13:47:49Z

+          "system-prompt.md",
+          async () => systemPrompt ?? DEFAULT_SYSTEM_PROMPT,
+        ),
+        ...(spec.dockerfile ? [FsTreeNode.fromTextFile("Dockerfile", spec.dockerfile)] : []),


Could we load or validate the Dockerfile before returning this tree? fromTextFile is not read until tree.write, after harness.json and system-prompt.md have already been written. A missing --dockerfile then leaves a partial harness directory behind even though the command fails.

agentcore-devx-automation · 2026-08-27T15:43:10Z

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

agentcore-devx-automation · 2026-08-27T15:51:52Z

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

notgitika · 2026-08-27T15:54:50Z

the schema was widened to ["strands", "none"] and the resolver is registered, but both flag declarations still say z.enum(["none"]).

so --framework strands is rejected and --help only lists none. is --template the only scope for this PR?

notgitika · 2026-08-27T15:56:52Z

+      return {
+        tree,
+        spec: {
+          harnesses: [{ name: parsed.name, path: join("app", parsed.name) }],


join("app", name) uses the platform separator, and the result is written into agentcore.json, which gets committed and shared.

on Windows that's "path": "app\\h1". if that is pulled down on macOS or Linux CI, it resolves that to a single filename with a backslash in it, and cdk deploy breaks for everyone except the author. can we always emit / and convert to platform separators only when touching the filesystem?

notgitika · 2026-08-27T15:58:18Z

there is an edge case that when scaffolding fails, cleanup runs rm -rf app/<name>. But that's the whole directory, not just the files this command created. If the user already had something there, it's gone.

is there a way to track the files actually written and delete only those. can be a followup

This might take some work to get right, lets do it as a follow-up.

sounds good to me!

agentcore-devx-automation · 2026-08-27T16:40:56Z

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

agentcore-devx-automation · 2026-08-27T16:46:01Z

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

agentcore-devx-automation · 2026-08-27T16:52:50Z

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

notgitika

thanks for addressing the comments!

jariy17 · 2026-08-26T21:19:34Z

@@ -0,0 +1,116 @@
+import os


I feel like we could use Strand's tools from folder feature to simplify this process here

tejaskash

Verified the requested fixes on the current head. Runtime additions now generate uv.lock, Python names are normalized before rendering, and missing Dockerfiles fail before any harness files are written. Focused tests and the full current-base merge validation passed.

github-actions Bot added the size/xl PR size: XL label Aug 25, 2026

agentcore-devx-automation Bot added agentcore-harness-reviewing AgentCore Harness review in progress claude-security-reviewing Claude Code /security-review in progress and removed agentcore-harness-reviewing AgentCore Harness review in progress labels Aug 25, 2026

agentcore-devx-automation Bot removed the claude-security-reviewing Claude Code /security-review in progress label Aug 25, 2026

Hweinstock mentioned this pull request Aug 25, 2026

refactor: align templates with old cli definition #2094

Merged

github-actions Bot added size/xl PR size: XL and removed size/xl PR size: XL labels Aug 25, 2026

agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Aug 25, 2026

agentcore-devx-automation Bot removed the claude-security-reviewing Claude Code /security-review in progress label Aug 25, 2026

github-actions Bot added size/xl PR size: XL and removed size/xl PR size: XL labels Aug 26, 2026

agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Aug 26, 2026

Hweinstock force-pushed the refactor-templates-pt2 branch from dd846e3 to a622914 Compare August 26, 2026 01:27

github-actions Bot added size/xl PR size: XL and removed size/xl PR size: XL labels Aug 26, 2026

agentcore-devx-automation Bot added claude-security-reviewing Claude Code /security-review in progress and removed claude-security-reviewing Claude Code /security-review in progress labels Aug 26, 2026

agentcore-devx-automation Bot removed the claude-security-reviewing Claude Code /security-review in progress label Aug 26, 2026

github-actions Bot added size/xl PR size: XL and removed size/xl PR size: XL labels Aug 26, 2026

Hweinstock mentioned this pull request Aug 27, 2026

fix(scaffold): allow certain template values to be overriden Hweinstock/agentcore-cli#5

Draft

aidandaly24 reviewed Aug 27, 2026

View reviewed changes

tejaskash reviewed Aug 27, 2026

View reviewed changes

Hweinstock added 4 commits August 27, 2026 15:09

fix(hello-world): reject non-HTTP protocol on template

b003cf9

docs(strands-python): remove non-existent command from readme

ca5224f

feat(template): install runtime dependencies in scaffolding

150a060

fix(python): normalize names before being sent to python templates

aba40d7

github-actions Bot added size/xl PR size: XL and removed size/xl PR size: XL labels Aug 27, 2026

agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Aug 27, 2026

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

fix(templates): validate dockerfile for harness/

d736054

github-actions Bot added size/xl PR size: XL and removed size/xl PR size: XL labels Aug 27, 2026

agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label Aug 27, 2026

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

notgitika reviewed Aug 27, 2026

View reviewed changes

fix(handlers): add strands as handler input

49f65b9

github-actions Bot added size/xl PR size: XL and removed size/xl PR size: XL labels Aug 27, 2026

fix(spec): avoid writing windows paths

917cc91

notgitika approved these changes Aug 27, 2026

View reviewed changes

aidandaly24 approved these changes Aug 27, 2026

View reviewed changes

jariy17 reviewed Aug 27, 2026

View reviewed changes

tejaskash approved these changes Aug 27, 2026

View reviewed changes


		In a new terminal, you can invoke that server with:

		`agentcore project invoke --dev "What can you do"`

Uh oh!

Conversation

Hweinstock commented Aug 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Problem

Solution

Testing / Verification

Future Work

Uh oh!

codecov-commenter commented Aug 25, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

agentcore-devx-automation Bot commented Aug 25, 2026

Uh oh!

agentcore-devx-automation Bot commented Aug 25, 2026

Uh oh!

agentcore-devx-automation Bot commented Aug 26, 2026

Uh oh!

agentcore-devx-automation Bot commented Aug 26, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

aidandaly24 Aug 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Hweinstock Aug 27, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

agentcore-devx-automation Bot commented Aug 27, 2026

Uh oh!

agentcore-devx-automation Bot commented Aug 27, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

agentcore-devx-automation Bot commented Aug 27, 2026

Uh oh!

agentcore-devx-automation Bot commented Aug 27, 2026

Uh oh!

agentcore-devx-automation Bot commented Aug 27, 2026

Uh oh!

notgitika left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

tejaskash left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

Hweinstock commented Aug 25, 2026 •

edited

Loading

codecov-commenter commented Aug 25, 2026 •

edited

Loading

aidandaly24 Aug 27, 2026 •

edited

Loading

Hweinstock Aug 27, 2026 •

edited

Loading