Kiro CLI 2.0 のヘッドレスモードを試してみた — API キー認証でエージェントを非対話実行

$ export KIRO_API_KEY=ksk_************************************
$ touch hoge
$ kiro-cli chat --no-interactive --trust-all-tools "List files in current directory"
All tools are now trusted (!). Kiro will execute tools without asking for confirmation.
Agents can sometimes do unexpected things so understand the risks.

Learn more at https://kiro.dev/docs/cli/chat/security/#using-tools-trust-all-safely

Reading directory: /Users/your-username/work/project (using tool: read, max depth: 0, max entries: 1000, excluding: defaults)
 ✓ Successfully read directory /Users/your-username/work/project (1 entries)
 - Completed in 0.0s

> There's just one file in the current directory:

- hoge (empty file, 0 bytes)

 ▸ Credits: 0.05 • Time: 6s

mkdir -p sample-app && cat > sample-app/app.py << 'EOF'
import os
import subprocess

def get_user_data(user_id):
    query = "SELECT * FROM users WHERE id = " + user_id
    result = subprocess.call("echo " + query, shell=True)
    password = "admin123"
    api_key = "AKIAIOSFODNN7EXAMPLE"
    return result

def process_file(filename):
    f = open(filename, "r")
    data = f.read()
    return data

# TODO: Add input validation
# TODO: Implement proper error handling
# FIXME: This function is too slow
def calculate(x, y):
    return x / y
EOF

# モデルを設定
$ kiro-cli settings chat.defaultModel claude-opus-4.6

# 設定を確認
$ kiro-cli settings chat.defaultModel
claude-opus-4.6 (global)

$ kiro-cli chat --no-interactive --trust-tools=read,grep \
  "sample-app/app.py のセキュリティ脆弱性をレビューしてください。各問題について重大度（Critical/High/Medium/Low）と修正方法を日本語で一覧にしてください。"

> まずファイルの内容を確認します。
Reading file: /Users/your-username/work/project/sample-app/app.py, all lines (using tool: read)
 ✓ Successfully read 487 bytes from /Users/your-username/work/project/sample-app/app.py
 - Completed in 0.0s

> 以下がセキュリティレビューの結果です。

### セキュリティ脆弱性一覧

| # | 脆弱性 | 重大度 | 該当行 | 説明 |
|---|--------|--------|--------|------|
| 1 | SQLインジェクション | Critical | L5 | user_id を文字列結合でクエリに埋め込んでおり、任意のSQL文を実行される危険がある |
| 2 | OSコマンドインジェクション | Critical | L6 | subprocess.call(..., shell=True) にユーザー入力を直接渡しており、任意のコマンドを実行される |
| 3 | ハードコードされたパスワード | High | L7 | password = "admin123" がソースコードに平文で埋め込まれている |
| 4 | ハードコードされたAPIキー | High | L8 | AWSアクセスキーがソースコードに平文で埋め込まれている。リポジトリ経由で漏洩するリスクが高い |
| 5 | ファイルリソースの未クローズ | Medium | L12 | open() の戻り値を close() せず、with 文も使っていないためリソースリークが発生する |
| 6 | パストラバーサル | Medium | L11-13 | filename を検証せずに open() しており、../../etc/passwd 等で任意ファイルを読み取られる |
| 7 | ゼロ除算 | Low | L19 | y が 0 の場合に ZeroDivisionError が発生する。入力バリデーションもエラーハンドリングもない |

### 修正方法

1. SQLインジェクション — パラメータ化クエリ（プレースホルダ）を使う。
   cursor.execute("SELECT * FROM users WHERE id = %s", (user_id,))

2. OSコマンドインジェクション — shell=True を使わず、引数をリストで渡す。
   そもそもこの echo 呼び出し自体が不要なら削除する。
   subprocess.call(["echo", query], shell=False)

3. ハードコードされたパスワード — 環境変数やシークレットマネージャから取得する。
   password = os.environ["APP_PASSWORD"]

4. ハードコードされたAPIキー — 同様に環境変数やAWS Secrets Manager等から取得する。
   api_key = os.environ["AWS_ACCESS_KEY_ID"]

5. ファイルリソースの未クローズ — with 文を使う。
   with open(filename, "r") as f:
       data = f.read()

6. パストラバーサル — ファイルパスを正規化し、許可ディレクトリ内かを検証する。
   safe_dir = "/allowed/path"
   real = os.path.realpath(filename)
   if not real.startswith(safe_dir):
       raise ValueError("不正なファイルパス")

7. ゼロ除算 — 入力を検証してからの除算、またはエラーハンドリングを追加する。
   def calculate(x, y):
       if y == 0:
           raise ValueError("除数に0は指定できません")
       return x / y

 ▸ Credits: 0.39 • Time: 22s

$ kiro-cli settings chat.defaultModel deepseek-3.2
$ kiro-cli chat --no-interactive --trust-tools=read,grep \
  "sample-app/app.py のセキュリティ脆弱性をレビューしてください。各問題について重大度（Critical/High/Medium/Low）と修正方法を日本語で一覧にしてください。"

> sample-app/app.py の内容を確認しました。以下のセキュリティ脆弱性を確認しました：

| 脆弱性 | 重大度 | 修正方法 |
|---------|--------|----------|
| SQLインジェクション | Critical | パラメータ化クエリの使用 |
| シェルコマンドインジェクション | Critical | シェルコマンドの使用を避ける |
| ハードコード認証情報 | High | 環境変数/シークレットマネージャー |
| ファイル操作 | Medium | パス検証と正規化 |
| 例外処理 | Medium | 適切なエラーハンドリング |
| ログ記録 | Low | セキュリティログの実装 |

 ▸ Credits: 0.13 • Time: 21s