Skip to content

feat: support --iteration-id in story update#2

Merged
studyzy merged 1 commit into
studyzy:mainfrom
huangdijia:codex/story-update-iteration-id
Apr 23, 2026
Merged

feat: support --iteration-id in story update#2
studyzy merged 1 commit into
studyzy:mainfrom
huangdijia:codex/story-update-iteration-id

Conversation

@huangdijia
Copy link
Copy Markdown
Contributor

@huangdijia huangdijia commented Apr 20, 2026

Summary

  • add --iteration-id to tapd story update
  • pass the iteration id through to UpdateStoryRequest
  • add tests and a README example

Background

  • the TAPD SDK already supports UpdateStoryRequest.IterationID, but the CLI did not expose it on tapd story update

Changes

  • register --iteration-id on the story update command
  • include IterationID when building the update request
  • verify help/spec output and request mapping in tests

Test Plan

  • run go test ./...

Risks

  • only supports setting or changing an iteration id
  • does not add a separate clear-iteration flow

#1

@huangdijia huangdijia marked this pull request as ready for review April 20, 2026 08:15
Copilot AI review requested due to automatic review settings April 20, 2026 08:15
Copy link
Copy Markdown

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 exposes TAPD SDK’s UpdateStoryRequest.IterationID through the CLI by adding --iteration-id to tapd story update, wiring it into the update request, and documenting/validating the behavior with tests and a README example.

Changes:

  • Register --iteration-id on tapd story update.
  • Pass flagIterationID into model.UpdateStoryRequest.IterationID.
  • Add tests for spec output / flag registration / request mapping, plus a README usage example.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File Description
internal/cmd/story.go Adds --iteration-id to story update and maps it to UpdateStoryRequest.IterationID.
internal/cmd/story_test.go Adds tests validating help/spec includes the flag and that update requests send iteration_id.
README.md Documents updating a story while switching iteration via --iteration-id.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread internal/cmd/story_test.go
@studyzy studyzy merged commit a883c59 into studyzy:main Apr 23, 2026
3 of 4 checks passed
@huangdijia huangdijia deleted the codex/story-update-iteration-id branch April 23, 2026 06:19
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.

3 participants