Skip to content

Conversation

@harleylrn
Copy link
Contributor

@harleylrn harleylrn commented Nov 20, 2025

Description

Due to rebranding of the Amazon Q to kiro-cli, new module was created

Type of Change

  • New module
  • New template
  • Bug fix
  • Feature/enhancement
  • Documentation
  • Other

Module Information

All defined in the README
Path: registry/harleylrn/modules/kiro-cli
New version: v1.0.0
Breaking change: [ ] Yes [x] No

Testing & Validation

  • Tests pass (bun test)
  • Code formatted (bun fmt)
  • Changes tested locally

Related Issues

Resolves #547

@harleylrn
Copy link
Contributor Author

@DevelopmentCats @matifali Please take a look on this one 😄
This is the new one instead of Amazon Q module.
Didn't want to change the existing one, so the naming will be confusing.

@harleylrn harleylrn changed the title feat: add kiro-cli module to harleylrn registry with updated tests feat: add kiro-cli module Nov 20, 2025
@harleylrn harleylrn force-pushed the feat/add-kiro-cli-module branch from a276538 to 5e3f3dd Compare November 20, 2025 13:41
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR introduces a new kiro-cli module to support the rebranded Amazon Q CLI (now called Kiro CLI). The module provides AI-powered coding assistance in Coder workspaces with MCP (Model Context Protocol) integration and task reporting capabilities.

Key Changes

  • New Terraform module for kiro-cli installation and configuration with customizable agent settings
  • Installation and startup scripts for automated deployment in Coder workspaces
  • Comprehensive test coverage with both TypeScript and HCL test suites
  • Complete documentation including authentication setup, usage examples, and troubleshooting

Reviewed Changes

Copilot reviewed 8 out of 10 changed files in this pull request and generated 14 comments.

Show a summary per file
File Description
registry/harleylrn/modules/kiro-cli/main.tf Core Terraform configuration defining variables, resources, and module integration with agentapi
registry/harleylrn/modules/kiro-cli/scripts/install.sh Installation script for downloading, extracting, and configuring kiro-cli with authentication
registry/harleylrn/modules/kiro-cli/scripts/start.sh Startup script that launches kiro-cli with proper environment configuration and agentapi integration
registry/harleylrn/modules/kiro-cli/templates/agent-config.json.tpl JSON template for default kiro-cli agent configuration including tools, resources, and MCP settings
registry/harleylrn/modules/kiro-cli/main.test.ts TypeScript test suite with 16 test cases covering basic usage, autonomous operation, and extended configuration
registry/harleylrn/modules/kiro-cli/kiro-cli.tftest.hcl Terraform native test suite validating resource creation, variable handling, and configuration scenarios
registry/harleylrn/modules/kiro-cli/README.md Comprehensive documentation covering prerequisites, authentication setup, configuration options, and usage examples
registry/harleylrn/README.md Author profile and module overview for the harleylrn namespace

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
harleylrn and others added 7 commits November 20, 2025 09:35
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@DevelopmentCats
Copy link
Contributor

Looks good! Ill give this a test and then a final review and we can get this merged.

@DevelopmentCats
Copy link
Contributor

@harleylrn It works great, but I would use the new tasks method in the tasks usage example as shown in the tasks template

Here is my example for testing the kiro-cli module for example

resource "coder_ai_task" "task" {
  count   = data.coder_task.me.enabled ? data.coder_workspace.me.start_count : 0
  app_id  = module.kiro-cli[count.index].task_app_id
}

data "coder_task" "me" {}

module "kiro-cli" {
  count           = data.coder_task.me.enabled ? data.coder_workspace.me.start_count : 0
  source          = "git::https://github.com/harleylrn/coder_registry.git//registry/harleylrn/modules/kiro-cli?ref=feat/add-kiro-cli-module"
  agent_id        = coder_agent.main.id
  workdir         = "/home/coder"
  auth_tarball    = "tarball-here"
  ai_prompt       = data.coder_task.me.prompt
  trust_all_tools = true
  # Task reporting configuration
  report_tasks    = true
  # Enable CLI app alongside web app
  pre_install_script = <<-EOT
    #!/bin/bash
    set -e
    sudo apt-get update
    sudo apt-get install -y zstd
  EOT
  cli_app              = true
  web_app_display_name = "Kiro CLI"
  cli_app_display_name = "Kiro CLI"
}

@harleylrn
Copy link
Contributor Author

Hi @DevelopmentCats
You mean the

data "coder_task" "me" {}

should be present in the example?

@DevelopmentCats
Copy link
Contributor

Hi @DevelopmentCats You mean the

data "coder_task" "me" {}

should be present in the example?

I personally would, since this needs to be defined in the template for the task prompt to be passed from the task ui to the template.

Tasks are now primarily only launched from the tasks-ui now.

@harleylrn
Copy link
Contributor Author

Make sense 😄
Will update that 😄

@harleylrn
Copy link
Contributor Author

@DevelopmentCats added 😄

@DevelopmentCats
Copy link
Contributor

@DevelopmentCats added 😄

I think I might have misunderstood you earlier.

You will need the resource as well as shown here:

resource "coder_ai_task" "task" {
  count   = data.coder_task.me.enabled ? data.coder_workspace.me.start_count : 0
  app_id  = module.kiro-cli[count.index].task_app_id
}

data "coder_task" "me" {}

and in the module you pass the prompt to the module from the task data source, and the count is defined based on the data source as well.

ai_prompt = data.coder_task.me.prompt
count = data.coder_task.me.enabled ? data.coder_workspace.me.start_count : 0

@harleylrn
Copy link
Contributor Author

@DevelopmentCats added 😄

I think I might have misunderstood you earlier.

You right, missed this part 😄 As it worked for me 😄
Updated.

@DevelopmentCats
Copy link
Contributor

Ill test one more time and then we can get this merged today.

@DevelopmentCats
Copy link
Contributor

@harleylrn Everything looks good with the functionality.

Could you please source the icon for the README and the module by adding it in this PR, and making a PR for coder/coder adding the icon to site/static/icons

@harleylrn
Copy link
Contributor Author

@DevelopmentCats

Could you please source the icon for the README and the module by adding it in this PR, and making a PR for coder/coder adding the icon to site/static/icons
The icon is already exists in the coder, kiro icon is there, I just reusing it.

@DevelopmentCats
Copy link
Contributor

DevelopmentCats commented Dec 8, 2025

@DevelopmentCats

Could you please source the icon for the README and the module by adding it in this PR, and making a PR for coder/coder adding the icon to site/static/icons
The icon is already exists in the coder, kiro icon is there, I just reusing it.

I would follow the pattern we use in all of our modules. We source the icon in the module, and coder/coder itself in all of our modules.

@harleylrn
Copy link
Contributor Author

I would follow the pattern we use in all of our modules. We source the icon in the module, and coder/coder itself in all of our modules.

I don't really understand what you mean. Example, may be? 😄 @DevelopmentCats

Comment on lines +1 to +8
---
display_name: "Michael Orlov"
bio: "Platform Engineer specializing in cloud infrastructure, DevOps automation, and developer experience tools"
avatar: "./.images/avatar.png"
github: "harleylrn"
support_email: "michael.orlov@gmail.com"
status: "community"
---
Copy link
Contributor

@DevelopmentCats DevelopmentCats Dec 8, 2025

Choose a reason for hiding this comment

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

Sorry I probably could have explained better.

@harleylrn

This is the icon reference I was talking about. This frontmatter is for registry.coder.com which does not have access to the icons from coder/coder which is why we must have a seperate icon here is an example from claude-code

---
display_name: Claude Code
description: Run the Claude Code agent in your workspace.
icon: ../../../../.icons/claude.svg
verified: true
tags: [agent, claude-code, ai, tasks, anthropic]
---

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Update Q Developer Module for AWS Announcements and Rename to Kiro CLI

3 participants