Skip to content

feat: show estimate in issues list, search, and read output#92

Open
lightstrike wants to merge 3 commits intoflipbit03:mainfrom
lightstrikelabs:feat/issues-estimate-output
Open

feat: show estimate in issues list, search, and read output#92
lightstrike wants to merge 3 commits intoflipbit03:mainfrom
lightstrikelabs:feat/issues-estimate-output

Conversation

@lightstrike
Copy link
Contributor

Summary

  • Add estimate: Option<f64> to IssueSummary, SearchSummary, and IssueDetail types
  • Add estimate column to IssueRow table output
  • Format estimate as integer when whole number (e.g. "3" not "3.0"), empty string when None
  • Estimate now appears in issues list, issues search, and issues read output

Test plan

  • cargo clippy --workspace -- -D warnings clean
  • cargo fmt --check clean
  • cargo test --workspace — all 64 offline tests pass
  • Smoke test: lineark issues list --limit 5 — estimate column visible
  • Smoke test: lineark issues search "test" -l 5 — estimate column visible
  • Smoke test: lineark issues read ENG-xxx — estimate field in detail output

Copy link
Owner

@flipbit03 flipbit03 left a comment

Choose a reason for hiding this comment

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

LGTM, from a functionality standpoint, but we should not delete these comments from unrelated pieces. Issue documented in #97

@flipbit03 flipbit03 added the safe-to-test Maintainer-approved: run online tests with API secrets label Feb 24, 2026
@github-actions github-actions bot removed the safe-to-test Maintainer-approved: run online tests with API secrets label Feb 24, 2026
@lightstrike lightstrike force-pushed the feat/issues-estimate-output branch from 4ad3a47 to 5ff5cb1 Compare February 28, 2026 21:53
@lightstrike
Copy link
Contributor Author

@flipbit03 Ready for review again — rebased onto main and reverted the unrelated comment deletions. All CI checks passing.

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.

2 participants