agent-skill-installer 0.1.5 → 0.1.7

package/bin/cli.js

@@ -3,6 +3,7 @@
 const fs = require("node:fs/promises");
 const path = require("node:path");
 const os = require("node:os");
+const readline = require("node:readline/promises");
 
 const REPO_ROOT = path.resolve(__dirname, "..");
 const SKILLS_SOURCE_DIR = path.join(REPO_ROOT, "skills");
@@ -21,7 +22,7 @@ Commands:
 
 Options:
   --dest <path>          Custom destination directory.
-  --force                Overwrite destination if a skill already exists.
+  --force                Overwrite destination if a skill already exists (no prompt).
   -h, --help             Show this help.
 `);
 }
@@ -112,6 +113,33 @@ async function listSkills() {
   }
 }
 
+async function confirmOverwrite(name, targetPath) {
+  if (!process.stdin.isTTY || !process.stdout.isTTY) {
+    throw new Error(
+      `Destination already exists for "${name}": ${targetPath} (non-interactive mode, use --force to overwrite)`
+    );
+  }
+
+  const rl = readline.createInterface({
+    input: process.stdin,
+    output: process.stdout
+  });
+
+  try {
+    const answer = (
+      await rl.question(
+        `Skill "${name}" already exists at ${targetPath}. Overwrite it? [y/N] `
+      )
+    )
+      .trim()
+      .toLowerCase();
+
+    return answer === "y" || answer === "yes";
+  } finally {
+    rl.close();
+  }
+}
+
 async function installSkills(rawArgs) {
   const parsed = parseInstallArgs(rawArgs);
   if (parsed.help) {
@@ -137,13 +165,16 @@ async function installSkills(rawArgs) {
     const to = path.join(parsed.dest, name);
     const exists = await dirExists(to);
 
-    if (exists && !parsed.force) {
-      throw new Error(
-        `Destination already exists for "${name}": ${to} (use --force to overwrite)`
-      );
+    if (exists && parsed.force) {
+      await fs.rm(to, { recursive: true, force: true });
     }
 
-    if (exists && parsed.force) {
+    if (exists && !parsed.force) {
+      const shouldOverwrite = await confirmOverwrite(name, to);
+      if (!shouldOverwrite) {
+        console.log(`Skipped "${name}" (kept existing skill).`);
+        continue;
+      }
       await fs.rm(to, { recursive: true, force: true });
     }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "agent-skill-installer",
-  "version": "0.1.5",
+  "version": "0.1.7",
   "description": "Install one or more Codex skills from this package via npx.",
   "bin": {
     "agent-skill-installer": "bin/cli.js"

package/skills/codex-session-daily-report/SKILL.md

@@ -7,45 +7,56 @@ Produce a daily report from local Codex sessions.
 
 ## Workflow
 
-1. Resolve target date.
-- If user says `today`, use an absolute date (e.g. `2026-03-02`).
+1. Resolve target date to absolute date `YYYY-MM-DD`.
+- If user says `today`, convert explicitly (for example `2026-03-02`).
+- If user says relative dates (`yesterday`, `tomorrow`), convert to concrete dates first.
 
-2. Read session files for that date.
+2. Extract `assistant final_answer` entries with bundled script.
 
 ```bash
-ls -1 ~/.codex/sessions/2026/03/02/*.jsonl
+python3 skills/codex-session-daily-report/scripts/extract_final_answers.py --date 2026-03-02
 ```
 
-3. Extract completed assistant outputs only (`final_answer`).
+3. If user asks for a custom session root, pass `--root`.
 
 ```bash
-for f in ~/.codex/sessions/2026/03/02/*.jsonl; do
-  jq -r 'select(.type=="response_item" and .payload.type=="message" and .payload.role=="assistant" and .payload.phase=="final_answer")
-    | .timestamp + "\t" + ((.payload.content // [])
-    | map(select(.type=="output_text") | .text)
-    | join(" ") | gsub("[\n\r\t]+";" ") | gsub(" +";" "))' "$f"
-done
+python3 skills/codex-session-daily-report/scripts/extract_final_answers.py \
+  --date 2026-03-02 \
+  --root ~/.codex/sessions
 ```
 
-4. Summarize by主题 and result.
-- Merge similar tasks into one topic line.
-- Keep key evidence: file paths, tests, and commit hashes if present.
-- Keep neutral language and avoid guessing.
+4. Summarize only completed outcomes from extracted lines.
+- Merge similar tasks into one topic.
+- Keep concrete evidence when present: file paths, test commands/results, commit hashes.
+- Keep neutral wording and avoid guessing missing facts.
 
-5. Output daily report only (not logs).
+5. Output report only, not raw logs, unless user explicitly asks for raw lines.
 
 ## Output Format
 
 Use this structure:
 
 1. 日期：`YYYY-MM-DD`
-2. 今日业务事项：按主题列 2-6 条
-3. 关键产出：代码改动/测试结论/提交哈希
-4. 风险与待办：未完成项、潜在误判点
+2. 统计快照：`会话文件 N`、`final_answer N`、`覆盖时段 HH:mm-HH:mm`
+3. 今日业务事项：按主题列 2-6 条，每条包含 `动作`、`结果`、`证据`
+4. 关键产出：按类型分组输出
+- 代码改动：关键文件路径
+- 验证结果：命令与结论
+- 提交记录：提交哈希与提交说明
+5. 风险与待办：每条包含 `风险/阻塞`、`影响`、`下一步`
+6. 数据缺口说明：若无会话文件、无 final_answer 或证据不足，明确说明影响范围
 
 ## Constraints
 
-- Prefer `assistant final_answer` over commentary.
+- Use `assistant final_answer` only; do not use `commentary` as completion evidence.
 - Do not output raw JSONL unless user explicitly asks.
 - Keep report concise and factual.
-- If no completed entries are found, state that clearly.
+- If no session files or no `final_answer` are found, state that clearly in the report.
+- Prefer explicit evidence fields over narrative paragraphs.
+- If evidence is missing for a topic, mark it as `证据不足` instead of guessing.
+
+## Resource
+
+- `skills/codex-session-daily-report/scripts/extract_final_answers.py`
+  - Input: `--date YYYY-MM-DD`, optional `--root`.
+  - Output: `timestamp<TAB>session_file<TAB>text`.

package/skills/codex-session-daily-report/agents/openai.yaml

@@ -1,4 +1,4 @@
 interface:
   display_name: "codex-session-daily-report"
-  short_description: "Generate daily reports from Codex session history."
-  default_prompt: "Use $codex-session-daily-report to read today's Codex sessions and output a concise daily report."
+  short_description: "Summarize Codex final answers into a daily report"
+  default_prompt: "Use $codex-session-daily-report to resolve a concrete target date, extract assistant final_answer entries from ~/.codex/sessions, and output a concise factual daily report."

package/skills/codex-session-daily-report/scripts/extract_final_answers.py

@@ -0,0 +1,116 @@
+#!/usr/bin/env python3
+"""Extract assistant final answers from Codex session JSONL files."""
+
+from __future__ import annotations
+
+import argparse
+import glob
+import json
+import os
+import re
+import sys
+from pathlib import Path
+
+WHITESPACE_RE = re.compile(r"\s+")
+
+
+def normalize_text(text: str) -> str:
+    return WHITESPACE_RE.sub(" ", text).strip()
+
+
+def parse_date(value: str) -> tuple[str, str, str]:
+    parts = value.split("-")
+    if len(parts) != 3 or not all(part.isdigit() for part in parts):
+        raise ValueError("日期格式必须为 YYYY-MM-DD")
+
+    year, month, day = parts
+    if len(year) != 4 or len(month) != 2 or len(day) != 2:
+        raise ValueError("日期格式必须为 YYYY-MM-DD")
+
+    return year, month, day
+
+
+def collect_session_files(root: str, date_value: str) -> list[str]:
+    year, month, day = parse_date(date_value)
+    pattern = os.path.join(root, year, month, day, "*.jsonl")
+    return sorted(glob.glob(pattern))
+
+
+def extract_records(file_path: str) -> list[str]:
+    records: list[str] = []
+    with Path(file_path).open("r", encoding="utf-8") as handle:
+        for line in handle:
+            if not line.strip():
+                continue
+
+            try:
+                item = json.loads(line)
+            except json.JSONDecodeError:
+                continue
+
+            if item.get("type") != "response_item":
+                continue
+
+            payload = item.get("payload") or {}
+            if payload.get("type") != "message":
+                continue
+            if payload.get("role") != "assistant":
+                continue
+            if payload.get("phase") != "final_answer":
+                continue
+
+            content = payload.get("content") or []
+            text_parts = [
+                part.get("text", "")
+                for part in content
+                if isinstance(part, dict) and part.get("type") == "output_text"
+            ]
+            text = normalize_text(" ".join(text_parts))
+            if not text:
+                continue
+
+            timestamp = item.get("timestamp", "")
+            session_file = os.path.basename(file_path)
+            records.append(f"{timestamp}\t{session_file}\t{text}")
+
+    return records
+
+
+def main() -> int:
+    parser = argparse.ArgumentParser(
+        description="提取指定日期的 assistant final_answer 内容",
+    )
+    parser.add_argument(
+        "--date",
+        required=True,
+        help="目标日期，格式 YYYY-MM-DD",
+    )
+    parser.add_argument(
+        "--root",
+        default="~/.codex/sessions",
+        help="sessions 根目录，默认 ~/.codex/sessions",
+    )
+    args = parser.parse_args()
+
+    root = os.path.expanduser(args.root)
+    files = collect_session_files(root, args.date)
+
+    if not files:
+        print(f"未找到会话文件: {args.date}", file=sys.stderr)
+        return 1
+
+    wrote = False
+    for file_path in files:
+        for record in extract_records(file_path):
+            print(record)
+            wrote = True
+
+    if not wrote:
+        print(f"未找到 final_answer: {args.date}", file=sys.stderr)
+        return 2
+
+    return 0
+
+
+if __name__ == "__main__":
+    sys.exit(main())