Skip to content

fix(cli): resolve --topic option by name across all commands#4

Merged
hideack merged 2 commits into
mainfrom
fix/topic-option-use-name
Apr 12, 2026
Merged

fix(cli): resolve --topic option by name across all commands#4
hideack merged 2 commits into
mainfrom
fix/topic-option-use-name

Conversation

@hideack

@hideack hideack commented Apr 12, 2026

Copy link
Copy Markdown
Owner

問題

ingest start --topic <名前> だけが findOrCreateTopic() を呼んでトピックを名前で解決していた。
それ以外のコマンド(decision, note, blocker, next, ingest calendar-start 等)は --topic に渡した値をそのままトピックIDとして使っていたため、事前に start でトピックを作っておかないと使えない、かつ内部IDを知っている必要があるという問題があった。

これが ingest topics が空になる主な原因でもあった。

修正内容

以下の全コマンドで --topic <name> をトピック名として受け取り、findOrCreateTopic() で解決(なければ自動生成)するよう統一した:

  • decision
  • note
  • blocker
  • next
  • ingest calendar-start
  • ingest calendar-end
  • ingest git-commit
  • ingest artifact-updated

Test plan

  • ingest decision --summary "..." --topic 新しいトピック名 でトピックが自動作成されることを確認
  • ingest topics に上記トピックが表示されることを確認
  • --topic を省略した場合は従来通り動作することを確認

🤖 Generated with Claude Code

hideack and others added 2 commits April 12, 2026 13:40
Previously only `ingest start` called findOrCreateTopic() by name;
other commands passed the option value directly as topic_id, requiring
users to know internal IDs. Now all commands accept a topic name and
resolve (or create) it consistently.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@hideack hideack merged commit 7aaf271 into main Apr 12, 2026
1 check passed
@hideack hideack deleted the fix/topic-option-use-name branch April 12, 2026 04:45
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.

1 participant