spovishun-skills 1.2.2 → 1.2.3

package/CHANGELOG.md

@@ -5,6 +5,43 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [1.2.3] — 2026-06-05
+
+Patch release. Fixes a contract drift between `scripts/notion/get-task.js` and the
+`notion-task-to-code` skill surfaced during Spovishun dogfooding: the skill documented
+a `<N-or-pageId>` shorthand that the script never actually supported.
+
+### Fixed
+
+- **`scripts/notion/get-task.js`** now accepts a bare numeric arg (`get-task.js 93`) and
+  normalizes it to `<projectPrefix()>-93` before resolving via the board query. Previously
+  a bare number fell through to `toDashed()` and Notion rejected it with
+  `path.page_id should be a valid uuid`.
+- **`scripts/notion/get-task.js`** gains the `text` output format (`--format=text`), matching
+  the parity already offered by `get-board.js` and `list-epics.js`. Default remains `json`.
+- **`skills/notion-task-to-code/SKILL.md`** Step 1d now spells out the three accepted forms
+  (`<prefix>-N` / `N` / `pageId`) and the available formats. The misleading `<N-or-pageId>`
+  shorthand is gone.
+
+### Changed
+
+- **`scripts/notion/get-task.js`** is now a dual CLI + module: it exports
+  `normalizeTaskArg`, `renderText`, `renderMd`, `resolvePageId`, and `VALID_FORMATS` for
+  unit testing while keeping its `main()` entrypoint behind a `require.main === module`
+  guard.
+
+### Added
+
+- **`test/scripts-notion-get-task.test.js`** — unit coverage for `normalizeTaskArg`
+  (bare-number rewrite, prefixed pass-through, pageId untouched, config-derived prefix),
+  for the regex contract with `lib/project-prefix.js`, for the new `text` format, and for
+  `renderText` body shape (with and without optional fields).
+
+### Manifests bumped
+
+- `notion-task-to-code` 1.0.1 → 1.0.2
+- `scripts/notion/package.json` 1.0.0 → 1.0.1
+
 ## [1.2.2] — 2026-06-04
 
 Patch release with two breaking-but-necessary changes:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "spovishun-skills",
-  "version": "1.2.2",
+  "version": "1.2.3",
   "description": "Portable Claude Code skills, agents, hooks and rules — bootstrapped from the Spovishun project. Installs into Claude Code, Codex, Windsurf and Cursor via per-assistant adapters.",
   "type": "module",
   "publishConfig": {

package/scripts/notion/get-task.js

@@ -10,7 +10,7 @@ const { toDashed } = require('./lib/page-id');
 const { resolveRelationIds, extractRelationIds } = require('./lib/resolve-relations');
 const { taskIdRegex, projectPrefix } = require('./lib/project-prefix');
 
-const VALID_FORMATS = ['json', 'md'];
+const VALID_FORMATS = ['json', 'md', 'text'];
 
 function parseArgs(argv) {
   let format = 'json';
@@ -23,7 +23,18 @@ function parseArgs(argv) {
   return { format, arg: positional[0] };
 }
 
-async function resolvePageId(token, arg) {
+// A bare task number ("93") is shorthand for "<prefix>-93". Without this
+// normalization the arg would fall through to toDashed() in resolvePageId
+// and be sent as an invalid Notion page_id, producing
+// "path.page_id should be a valid uuid".
+function normalizeTaskArg(arg) {
+  if (typeof arg !== 'string') return arg;
+  if (/^\d+$/.test(arg)) return `${projectPrefix()}-${arg}`;
+  return arg;
+}
+
+async function resolvePageId(token, rawArg) {
+  const arg = normalizeTaskArg(rawArg);
   if (taskIdRegex().test(arg)) {
     const result = await http.post(token, `/v1/databases/${constants.DATABASE_ID}/query`, {
       filter: { property: 'Name', title: { contains: arg } },
@@ -56,6 +67,22 @@ function renderMd(task) {
   return parts.join('\n\n');
 }
 
+function renderText(task) {
+  const lines = [task.title];
+  const meta = [task.status, task.priority, task.branch].filter(Boolean).join(' | ');
+  if (meta) lines.push(meta);
+  if (task.epic) lines.push(`Epic: ${task.epic.title ?? task.epic.id}`);
+  if (task.blockedBy && task.blockedBy.length > 0) {
+    const list = task.blockedBy.map(b => `  - ${b.title ?? b.id}`).join('\n');
+    lines.push(`Blocked by:\n${list}`);
+  }
+  if (task.content) {
+    lines.push('');
+    lines.push(task.content);
+  }
+  return lines.join('\n');
+}
+
 async function main() {
   const token = loadToken();
   if (!token) {
@@ -71,7 +98,8 @@ async function main() {
   const { format, arg } = parseArgs(process.argv.slice(2));
 
   if (!arg) {
-    process.stderr.write(`Usage: get-task.js <pageId | ${projectPrefix()}-N> [--format=json|md]\n`);
+    const p = projectPrefix();
+    process.stderr.write(`Usage: get-task.js <${p}-N | N | pageId> [--format=json|md|text]\n`);
     process.exit(1);
   }
 
@@ -114,12 +142,20 @@ async function main() {
 
   if (format === 'md') {
     process.stdout.write(renderMd(task) + '\n');
+  } else if (format === 'text') {
+    process.stdout.write(renderText(task) + '\n');
   } else {
     process.stdout.write(JSON.stringify(task) + '\n');
   }
 }
 
-main().catch(err => {
-  process.stderr.write(`Error: ${err.message}\n`);
-  process.exit(1);
-});
+// Exported for unit tests. The CLI entry runs main() unconditionally below;
+// the require.main guard keeps tests from triggering it.
+module.exports = { normalizeTaskArg, renderMd, renderText, resolvePageId, VALID_FORMATS };
+
+if (require.main === module) {
+  main().catch(err => {
+    process.stderr.write(`Error: ${err.message}\n`);
+    process.exit(1);
+  });
+}

package/scripts/notion/package.json

@@ -1,6 +1,6 @@
 {
   "name": "spovishun-skills-notion-cli",
-  "version": "1.0.0",
+  "version": "1.0.1",
   "private": true,
   "type": "commonjs",
   "description": "Notion CLI helpers shipped by the spovishun-skills plugin. Installed into consumer projects under .claude/scripts/notion/. Skill bodies invoke these as `node .claude/scripts/notion/<script>.js`."

package/skills/notion-task-to-code/SKILL.md

@@ -17,9 +17,13 @@ git rev-parse --abbrev-ref HEAD
 
 **1d.** Otherwise (standalone invocation or no cache) → fetch from Notion:
 ```
-node .claude/scripts/notion/get-task.js <N-or-pageId>
+node .claude/scripts/notion/get-task.js <{{PROJECT_PREFIX}}-N | N | pageId> [--format=json|md|text]
 ```
-Accepts `{{PROJECT_PREFIX}}-19`, bare `19`, or a 32-char compact pageId.
+- `{{PROJECT_PREFIX}}-19` — fully-qualified task id (board lookup by name).
+- `19` — bare number; resolves to `{{PROJECT_PREFIX}}-19` automatically.
+- 32-char compact (or dashed) Notion `pageId` — direct fetch.
+
+Default `--format=json`. Use `md` for a rendered markdown card or `text` for a plain human-readable summary.
 
 ### Step 2: Fetch CLAUDE.md (targeted)
 ```

package/skills/notion-task-to-code/manifest.yaml

@@ -1,5 +1,5 @@
 id: notion-task-to-code
-version: 1.0.1
+version: 1.0.2
 category: stack-specific
 requires:
   - notion