pi-copy 0.1.1 → 0.2.0

package/README.md

@@ -1,22 +1,23 @@
 # pi-copy
 
-A pi extension that copies **one part** of the last answer so it pastes cleanly: a code block exactly as written (a long command continued with `\` pastes into a terminal and runs), or the prose as plain text (one line per paragraph, ready for an email).
+A pi extension that copies **one part** of the last answer so it pastes cleanly: a code block exactly as written (a long command continued with `\` pastes into a terminal and runs), a quoted message on its own, or the prose as plain text (one line per paragraph, ready for an email).
 
 Pi's own `/copy` (Ctrl+X) copies the whole last answer as markdown source, fences and marks included; drag-selecting copies the rendered screen, wraps and borders included. pi-copy reads the answer's source text from the session instead, never the screen.
 
 ## Use
 
 - `/copy-part` opens a picker of the answer's parts. Enter copies the chosen part; Escape copies nothing.
-- **alt+c** does the same, except that when the answer has exactly one code block it copies that block at once.
+- **alt+c** does the same, except that it copies at once when the answer has exactly one code block, or no code block and exactly one quote.
 - After a copy, a one-line notice names what was copied and its size, such as `Copied code block (1 line, 148 chars)`. With no answer yet it says `No answer to copy yet.`
 
 ## The parts
 
 - **Each code block**, in order, labelled with its language and its first line. Copied as its exact text: no fences, no indent, no trailing newline. A block inside a list item counts too.
+- **Each quote**, in order, labelled with its first line: every blockquote not inside another, a blockquote in a list item included. Copied as plain text by the prose rules below, without the `>` marks, with any code block inside it in place. A message an answer writes for someone else, such as one under "Message for Dana:", is usually a quote.
 - **The prose**: everything outside the code blocks, as plain text. Paragraphs are one line each, separated by a blank line; bold, italic, inline code and strike marks are removed; a link becomes `text (url)`, or just the address when the two are the same; headings are their text on a line of their own; list items keep `- ` or `1. ` with two spaces per nesting level; a blockquote loses its `>`; a table is copied as its markdown source rows.
 - **All**: the whole answer, the prose as above with each code block's text in place.
 
-An answer with no code blocks has only the prose part; one with no prose has only its code blocks.
+An answer with no code blocks or quotes has only the prose part; one with no prose has only its code blocks.
 
 Only the last answer is read. An earlier answer can still be copied whole from `/tree` with Ctrl+X.
 
@@ -28,7 +29,7 @@ Only the last answer is read. An earlier answer can still be copied whole from `
 
 ## Tests
 
-`node --test src/*.test.ts`: the parts and each part's copied text (continued commands, a block in a list, paragraphs, inline marks, links, headings, nested lists, blockquotes, tables, the notice), and the wiring against a fake pi (alt+c's one-block shortcut, the picker, Escape, no answer, an aborted answer skipped).
+`node --test src/*.test.ts`: the parts and each part's copied text (continued commands, a block in a list, paragraphs, inline marks, links, headings, nested lists, blockquotes, quote parts, tables, the notice), and the wiring against a fake pi (alt+c's one-block and one-quote shortcuts, the picker, Escape, no answer, an aborted answer skipped).
 
 ## Install
 

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "pi-copy",
-  "version": "0.1.1",
-  "description": "Copy one part of the last pi answer from its source text: a code block exactly as written, or the prose as plain text, through /copy-part and alt+c.",
+  "version": "0.2.0",
+  "description": "Copy one part of the last pi answer from its source text: a code block exactly as written, a quoted message, or the prose as plain text, through /copy-part and alt+c.",
   "type": "module",
   "license": "MIT",
   "repository": {

package/src/index.test.ts

@@ -66,6 +66,37 @@ test("alt+c with one code block copies it without the picker", async () => {
   assert.deepEqual(f.notices, ["Copied code block (1 line, 41 chars)"]);
 });
 
+const salomon =
+  "Message for Salomon:\n\n> Dev Fetch’s CRM now works. Please walk a pitch deck through Make it.\n\nSaved and pushed as `d31765e`.";
+
+test("alt+c with one quote and no code block copies the quote without the picker", async () => {
+  const f = makeFixture([assistant(salomon)]);
+
+  await f.altC();
+
+  assert.equal(f.selects.length, 0);
+  assert.deepEqual(f.copied, ["Dev Fetch’s CRM now works. Please walk a pitch deck through Make it."]);
+  assert.deepEqual(f.notices, ["Copied quote (1 line, 68 chars)"]);
+});
+
+test("alt+c with one code block and one quote copies the code block", async () => {
+  const f = makeFixture([assistant(`${oneBlock}\n\n> Tell Dana it is tagged.`)]);
+
+  await f.altC();
+
+  assert.equal(f.selects.length, 0);
+  assert.deepEqual(f.copied, ["git tag pi-copy-v0.1.0 && git push --tags"]);
+});
+
+test("alt+c with two quotes and no code block opens the picker", async () => {
+  const f = makeFixture([assistant("For Dana:\n\n> Deck is ready.\n\nFor Sam:\n\n> Room is booked.")], (labels) => labels[1]);
+
+  await f.altC();
+
+  assert.equal(f.selects.length, 1);
+  assert.deepEqual(f.copied, ["Room is booked."]);
+});
+
 test("alt+c with two code blocks opens the picker and copies the choice", async () => {
   const f = makeFixture([assistant(twoBlocks)], (labels) => labels[1]);
 

package/src/index.ts

@@ -1,10 +1,11 @@
 // pi-copy — copy one part of the last answer: a code block exactly as
-// written, or the prose as plain text.
+// written, a quoted message, or the prose as plain text.
 //
 // The answer is read from the session (the same message pi's /copy uses),
 // never from the screen, so nothing copied carries pi's wrapping or borders.
 // `/copy-part` always opens a picker of the parts; alt+c copies the one code
-// block at once when there is exactly one, and opens the picker otherwise.
+// block at once when there is exactly one, or the one quote when there is no
+// code block, and opens the picker otherwise.
 // The copy goes through pi's own copyToClipboard, which also reaches the
 // local clipboard from tmux and over SSH (OSC 52).
 //
@@ -54,9 +55,11 @@ export function createCopy(pi: any, deps: Deps = {}): void {
     }
     const parts = splitParts(text);
     const code = parts.filter((p) => p.kind === "code");
+    const quote = parts.filter((p) => p.kind === "quote");
+    const only = code.length === 1 ? code[0] : code.length === 0 && quote.length === 1 ? quote[0] : undefined;
     let part: Part | undefined;
-    if (!alwaysPick && code.length === 1) {
-      part = code[0];
+    if (!alwaysPick && only) {
+      part = only;
     } else {
       const label = await ctx.ui.select(
         "Copy which part?",
@@ -74,7 +77,7 @@ export function createCopy(pi: any, deps: Deps = {}): void {
   }
 
   pi.registerCommand("copy-part", {
-    description: "Copy one part of the last answer: a code block as written, or the prose as plain text",
+    description: "Copy one part of the last answer: a code block as written, a quoted message, or the prose as plain text",
     handler: async (_args: string, ctx: any) => run(ctx, true),
   });
 

package/src/parts.test.ts

@@ -156,3 +156,72 @@ test("notice counts lines of a multi-line part", () => {
 
   assert.equal(notice(parts[0]!), "Copied prose (3 lines, 31 chars)");
 });
+
+// The answer that copied whole: a message for someone else in a blockquote, no code block.
+const salomonQuote =
+  "Dev Fetch’s CRM now works. Please use your existing local dashboard against dev Fetch to walk a pitch deck and questionnaire through Make it, open the results, and verify Made with Prism. Also check Change before confirming. No separately hosted dashboard needed.";
+const salomon = [
+  "Updated ASK3: no Amplify branch, just the local walkthrough against dev Fetch.",
+  "",
+  "Message for Salomon:",
+  "",
+  "> Dev Fetch’s CRM now works. Please use your existing local dashboard against dev Fetch to walk a pitch deck",
+  "> and questionnaire through Make it, open the results, and verify Made with Prism. Also check Change before",
+  "> confirming. No separately hosted dashboard needed.",
+  "",
+  "Saved and pushed as `d31765e`.",
+].join("\n");
+
+test("a blockquote is its own part, copied as one line without the > marks", () => {
+  const parts = splitParts(salomon);
+
+  assert.deepEqual(
+    parts.map((p) => p.kind),
+    ["quote", "prose", "all"],
+  );
+  assert.equal(parts[0]!.label, "Quote 1: Dev Fetch’s CRM now works. Please use your existing local d…");
+  assert.equal(parts[0]!.text, salomonQuote);
+  assert.equal(notice(parts[0]!), "Copied quote (1 line, 263 chars)");
+});
+
+test("the prose part still holds the whole answer, quote included", () => {
+  assert.equal(
+    prose(salomon),
+    `Updated ASK3: no Amplify branch, just the local walkthrough against dev Fetch.\n\nMessage for Salomon:\n\n${salomonQuote}\n\nSaved and pushed as d31765e.`,
+  );
+});
+
+test("a blockquote inside a list item is a part", () => {
+  const answer = "1. Send Dana this:\n\n   > The deck is ready for Thursday.\n\n2. Then book the room.";
+
+  const quotes = splitParts(answer).filter((p) => p.kind === "quote");
+
+  assert.deepEqual(
+    quotes.map((p) => p.text),
+    ["The deck is ready for Thursday."],
+  );
+});
+
+test("a blockquote inside a blockquote is one part with the outer one", () => {
+  const answer = "Forward this:\n\n> Hi Dana,\n>\n> > Can we move to Friday?\n>\n> Friday works.";
+
+  const quotes = splitParts(answer).filter((p) => p.kind === "quote");
+
+  assert.deepEqual(
+    quotes.map((p) => p.text),
+    ["Hi Dana,\n\nCan we move to Friday?\n\nFriday works."],
+  );
+});
+
+test("a code block inside a quote is copied in place and is still its own part", () => {
+  const answer = "Tell Sam:\n\n> Run this to log in:\n>\n> ```sh\n> aws sso login --profile fetch\n> ```\n>\n> Then retry the deploy.";
+
+  const parts = splitParts(answer);
+
+  assert.deepEqual(
+    parts.map((p) => p.kind),
+    ["code", "quote", "prose", "all"],
+  );
+  assert.equal(parts[0]!.text, "aws sso login --profile fetch");
+  assert.equal(parts[1]!.text, "Run this to log in:\n\naws sso login --profile fetch\n\nThen retry the deploy.");
+});

package/src/parts.ts

@@ -6,10 +6,12 @@
 // its text is marked's: fences and indent removed, no trailing newline.
 // Prose is rendered to plain text here: one line per paragraph, inline marks
 // dropped, links as `text (url)`, list markers kept, tables as source rows.
+// Each blockquote is a part too (a message for someone else, usually),
+// rendered like prose with its code in place.
 
 import { Lexer, type Token, type Tokens } from "marked";
 
-export type Part = { kind: "code" | "prose" | "all"; label: string; text: string };
+export type Part = { kind: "code" | "quote" | "prose" | "all"; label: string; text: string };
 
 const PREVIEW = 60;
 
@@ -29,6 +31,16 @@ function codeBlocks(tokens: Token[], out: Tokens.Code[] = []): Tokens.Code[] {
   return out;
 }
 
+// Every blockquote not inside another, in source order, wherever it sits.
+function quotes(tokens: Token[], out: Tokens.Blockquote[] = []): Tokens.Blockquote[] {
+  for (const token of tokens) {
+    if (token.type === "blockquote") out.push(token as Tokens.Blockquote);
+    else if (token.type === "list") for (const item of (token as Tokens.List).items) quotes(item.tokens, out);
+    else if ("tokens" in token && Array.isArray(token.tokens)) quotes(token.tokens, out);
+  }
+  return out;
+}
+
 // A code block's exact text, without the trailing newlines a blank line before the fence leaves.
 function codeText(code: Tokens.Code): string {
   return code.text.replace(/\n+$/, "");
@@ -132,6 +144,10 @@ export function splitParts(markdown: string): Part[] {
     label: `Code block ${index + 1}${code.lang ? ` (${code.lang})` : ""}: ${preview(code.text)}`,
     text: codeText(code),
   }));
+  quotes(tokens).forEach((quote, index) => {
+    const text = blocks(quote.tokens, true).join("\n\n");
+    if (text !== "") parts.push({ kind: "quote", label: `Quote ${index + 1}: ${preview(text)}`, text });
+  });
   const prose = blocks(tokens, false).join("\n\n");
   if (prose === "") return parts;
   parts.push({ kind: "prose", label: `Prose: ${preview(prose)}`, text: prose });
@@ -139,7 +155,7 @@ export function splitParts(markdown: string): Part[] {
   return parts;
 }
 
-const NOUN: Record<Part["kind"], string> = { code: "code block", prose: "prose", all: "the whole answer" };
+const NOUN: Record<Part["kind"], string> = { code: "code block", quote: "quote", prose: "prose", all: "the whole answer" };
 
 export function notice(part: Part): string {
   const lines = part.text.split("\n").length;