Skip to content

feat: comments update, resolve, and unresolve commands#108

Merged
flipbit03 merged 5 commits intoflipbit03:mainfrom
lightstrikelabs:feat/comments-update-resolve
Mar 7, 2026
Merged

feat: comments update, resolve, and unresolve commands#108
flipbit03 merged 5 commits intoflipbit03:mainfrom
lightstrikelabs:feat/comments-update-resolve

Conversation

@lightstrike
Copy link
Copy Markdown
Contributor

@lightstrike lightstrike commented Mar 1, 2026

Summary

  • Add commentUpdate, commentResolve, commentUnresolve mutations to operations.toml and regenerate SDK
  • New CLI subcommands:
    • lineark comments update <UUID> --body TEXT — update a comment's body
    • lineark comments resolve <UUID> [--resolving-comment UUID] — resolve a comment thread
    • lineark comments unresolve <UUID> — unresolve a comment thread
  • Extend CommentRef lean type with resolved_at for meaningful resolve/unresolve output
  • Update lineark usage reference

Test plan

  • make check — clean
  • make test — all offline tests pass (191)
  • SDK online tests pass: 38/38 including comment_update_changes_body, comment_resolve_and_unresolve
  • CLI online tests pass: full lifecycle (create→update→resolve→unresolve→delete), --resolving-comment flag
  • All tests migrated to TeamGuard pattern for self-containment

lightstrike and others added 4 commits February 28, 2026 18:37
Add three GraphQL mutations to complete the comment lifecycle:
- commentUpdate: update a comment's body
- commentResolve: resolve a comment thread (with optional resolving comment)
- commentUnresolve: unresolve a previously resolved comment thread

Includes CLI subcommands, usage reference, 9 offline tests, 2 SDK online
tests, and 2 CLI online tests.
…resolve

# Conflicts:
#	crates/lineark-sdk/tests/online.rs
#	crates/lineark/tests/offline.rs
#	crates/lineark/tests/online.rs
@flipbit03 flipbit03 marked this pull request as ready for review March 7, 2026 15:59
Brings in TeamGuard infrastructure from flipbit03#119. Migrates comment and
find-branch tests from teams[0] to create_test_team() for full
self-containment. The flipbit03#105 blocker no longer applies.
@flipbit03 flipbit03 added the safe-to-test Maintainer-approved: run online tests with API secrets label Mar 7, 2026
@github-actions github-actions Bot removed the safe-to-test Maintainer-approved: run online tests with API secrets label Mar 7, 2026
@flipbit03 flipbit03 merged commit ba7ce7f into flipbit03:main Mar 7, 2026
9 checks passed
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