shapeup-cli 0.8.1 → 0.10.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cc0a67b4cc8086fc7ba5a2984d6be2b2eab9041ce4efe344e3030840e4cb6c21
-  data.tar.gz: 2ae7641ddd1b78c4777200191c168e9d7b461785f552b598715864cf9555e4b9
+  metadata.gz: f7f49a40c881c3e40aa1749a419a41139d02a5c3066113e851efecf544cddc84
+  data.tar.gz: baa1415c2d57ea6aeeb74c4562e67773041ddd72ad9d27667bf34756faca8e56
 SHA512:
-  metadata.gz: 508c56acc3c2b28de638e76ac8ec7d5771daca19a5c083392cb876b4b2a8f6293c45205b158a55507ad7509f9be152b83c57a61cd2a90678967c9697e680c030
-  data.tar.gz: c39c3a4ad91d481aee6cc60ed58eb30b21396323ef0de3f619e9f38564e63b67b9860dd4d445301354fee55217e107aa9cdf86d817baa083677b8e6df9b19e08
+  metadata.gz: 83fb9f6f9978710170a847e270469de2b7b0d57381adb4751a0748db279477e6780d2ca87176f1f787a06cd89c5594e7cd65533c773d6b7fb12b7d0f3ee3747d
+  data.tar.gz: '098b476ae487d54975b7726227407462452da622fb5a7b188de2bd72d42a902dfc10f2fe474874eda44069b2014b5ea7d725a021e8cd988e90c7f8fa009faf11'

data/lib/shapeup_cli/commands/base.rb

@@ -100,6 +100,14 @@ module ShapeupCli
           value
         end
 
+        def extract_options(flag)
+          values = []
+          while (value = extract_option(flag))
+            values << value
+          end
+          values
+        end
+
         # Get the first positional arg (not a flag)
         def positional_arg(position = 0)
           positionals = @remaining.reject { |a| a.start_with?("--") }

data/lib/shapeup_cli/commands/comments.rb

@@ -20,7 +20,8 @@ module ShapeupCli
             { name: "pitch", type: "string", usage: "Pitch ID" },
             { name: "scope", type: "string", usage: "Scope ID" },
             { name: "task", type: "string", usage: "Task ID" },
-            { name: "parent", type: "string", usage: "Comment ID to reply under" }
+            { name: "parent", type: "string", usage: "Comment ID to reply under" },
+            { name: "attach", type: "string", usage: "Image to show under the comment (repeatable). Local ![description](path) images in the text are attached too" }
           ],
           examples: [
             "shapeup comments list --issue 42",
@@ -62,10 +63,12 @@ module ShapeupCli
         def add
           type, id = resolve_commentable
           parent = extract_option("--parent")
-          text = positional_arg(1) || abort('Usage: shapeup comments add --issue <id> [--parent <comment_id>] "Comment text"')
+          attach_paths = extract_options("--attach")
+          text = positional_arg(1) || abort('Usage: shapeup comments add --issue <id> [--parent <comment_id>] [--attach <image>] "Comment text"')
 
-          arguments = { commentable_type: type, commentable_id: id.to_s, content: text }
+          arguments = { commentable_type: type, commentable_id: id.to_s, content: place_local_images(text) }
           arguments[:parent_id] = parent.to_s if parent
+          arguments[:attachments] = attach_paths.map { |path| upload_image(path, nil) } if attach_paths.any?
           result = call_tool("create_comment", **arguments)
 
           render result,
@@ -79,7 +82,7 @@ module ShapeupCli
           comment_id = positional_arg(1) || abort('Usage: shapeup comments edit <comment_id> "New text"')
           text = positional_arg(2) || abort('Usage: shapeup comments edit <comment_id> "New text"')
 
-          result = call_tool("update_comment", comment_id: comment_id.to_s, content: text)
+          result = call_tool("update_comment", comment_id: comment_id.to_s, content: place_local_images(text))
 
           render result, summary: "Comment ##{comment_id} updated"
         end
@@ -103,6 +106,31 @@ module ShapeupCli
           render result, summary: "Reacted #{emoji} to comment ##{comment_id}"
         end
 
+        IMAGE_TYPES = { ".png" => "image/png", ".jpg" => "image/jpeg", ".jpeg" => "image/jpeg", ".gif" => "image/gif", ".webp" => "image/webp" }.freeze
+        IMAGE_MAX_SIZE = 10 * 1024 * 1024
+        MARKDOWN_IMAGE = /!\[([^\]]*)\]\(([^)\s]+)(?:\s+"([^"]*)")?\)/
+
+        def place_local_images(text)
+          text.gsub(MARKDOWN_IMAGE) do |markdown|
+            alt, path, caption = $1, $2, $3
+            next markdown if path.match?(%r{\A[a-z][a-z0-9+.-]*:}i) || !File.file?(path)
+
+            uploaded = upload_image(path, alt)
+            %(![#{uploaded[:alt]}](attachment:#{uploaded[:attachment]}#{%( "#{caption}") if caption}))
+          end
+        end
+
+        def upload_image(path, alt)
+          abort "No such file: #{path}" unless File.file?(path)
+          content_type = IMAGE_TYPES[File.extname(path).downcase] or abort "Only PNG, JPEG, GIF and WebP images can be attached: #{path}"
+          abort "Images can be at most 10 MB: #{path}" if File.size(path) > IMAGE_MAX_SIZE
+
+          data = [ File.binread(path) ].pack("m0")
+          uploaded = Output.extract_data(call_tool("upload_attachment", filename: File.basename(path), content_type: content_type, data: data))
+
+          { attachment: uploaded["attachment"], alt: alt.to_s.empty? ? File.basename(path, ".*").tr("-_", " ") : alt }
+        end
+
         def resolve_commentable
           issue = extract_option("--issue")
           pitch = extract_option("--pitch")

data/lib/shapeup_cli/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module ShapeupCli
-  VERSION = "0.8.1"
+  VERSION = "0.10.0"
 end

data/skills/shapeup/SKILL.md

@@ -222,6 +222,7 @@ Branch on `retryable` to decide retry-vs-abandon; `false` means "no known reason
 | Watch the inbox as JSON lines | `SHAPEUP_PROFILE=<agent> shapeup connect` |
 | Reply under a comment | `shapeup comments add --issue <id> --parent <comment_id> "Reply"` |
 | React to a comment | `shapeup comments react <comment_id> 👀` |
+| Attach screenshots to a comment | `shapeup comments add --issue <id> "Text ![What it shows](./shot.png \"Caption\")"` (stays where you put it; images on consecutive lines sit side by side) or `--attach shot.png` (added at the end) |
 
 ## Common Workflows
 

data/skills/shapeup-connect/SKILL.md

@@ -142,13 +142,38 @@ Give the subagent this, filled in:
 >    commit there. Never push, deploy, merge or delete branches unless the owner said so in this
 >    session.
 > 4. If the ask is unclear or you are blocked, ask on the issue rather than guess.
-> 5. Reply once, under the comment that asked:
+> 5. If the work changes anything people see, compare it the way a careful reviewer would:
+>    - From your diff, list the screens and states it touches, including the ones it could break.
+>    - Run the main branch beside yours: check main out into a git worktree outside this
+>      repository and start it on a second port, so both are up at once. How to start and sign in
+>      to the app should be in the project's CLAUDE.md or README; if it isn't, ask on the issue.
+>    - With whatever browser tool you have, capture each screen in both, at the same size and with
+>      the same content, plus a phone width and dark mode where they matter. Save PNGs outside the
+>      repository, named for what they show.
+>    - Stop the second server and remove the worktree when you're done.
+> 6. Reply once, under the comment that asked:
 >    `shapeup --profile <profile> comments add --issue <issue.id> --parent <reply_to.comment_id> "<reply>"`
->    (leave out `--parent` when `reply_to.comment_id` is null). Start with `@<requester.name>`.
->    Say what changed, the branch and commit, how you checked it, and anything still open.
->    Keep it short and plain.
-> 6. Then take it out of the inbox: `shapeup --profile <profile> inbox done <id>`.
-> 7. Report back in one line.
+>    (leave out `--parent` when `reply_to.comment_id` is null). Start with `@<requester.name>`,
+>    then say what changed, the branch and commit, and how you checked it. Keep it plain.
+>
+>    When you have screenshots, write the rest like a review: a `## heading` per area, a line on
+>    what changed there, then its before and after as local Markdown images on consecutive lines,
+>    so they sit side by side:
+>
+>    ```markdown
+>    ## Toolbar
+>    Looks the same. Dropdowns now open with focus on the first item.
+>
+>    ![Toolbar on main, dropdown open](/tmp/shots/toolbar-before.png "Before")
+>    ![Toolbar on this branch, dropdown open with the first item focused](/tmp/shots/toolbar-after.png "After")
+>    ```
+>
+>    The CLI uploads each image where it stands. The words in brackets are its description for
+>    people who can't see it, so say what it shows; the quoted word is its caption. End with a
+>    `## Worth a look` section for anything that seems off, and mention whoever the requester asked
+>    you to hand over to.
+> 7. Then take it out of the inbox: `shapeup --profile <profile> inbox done <id>`.
+> 8. Report back in one line.
 
 ## Rules
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: shapeup-cli
 version: !ruby/object:Gem::Version
-  version: 0.8.1
+  version: 0.10.0
 platform: ruby
 authors:
 - ShapeUp