td-ai-tools 1.4.0 → 1.4.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "td-ai-tools",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "description": "Install agent skills and packs into your project",
   "type": "module",
   "scripts": {

package/skills/barrage/SKILL.md

@@ -27,9 +27,11 @@ other repo installed).
 ## Input: the expected Basecamp item shape
 
 This skill is designed for Basecamp items whose **titles carry a trailing `[Nh]`
-estimate suffix** — e.g. `Build the checkout page [2h]` or `Fix nav [1.5h]`. That
-convention comes from an estimation workflow (the **td-augury** generator produces
-titles in exactly this shape), but nothing here requires that tool: any Basecamp
+estimate suffix** — e.g. `Build the checkout page [2h]`, `Fix nav [1.5h]` or
+`[KS] ContainedBanner [0.25hrs]`. The unit may be written `h`, `hr`, `hrs`, `hour`
+or `hours` (any case). That convention comes from an estimation workflow (the
+**td-augury** generator produces titles in exactly this shape), but nothing here
+requires that tool: any Basecamp
 todo or card works. Titles *without* a suffix are used verbatim; titles *with* one
 have it stripped from the task id by default (see `--keep-estimate`).
 
@@ -45,7 +47,7 @@ Each Basecamp item becomes one td-barrage task:
 
 | td-barrage field | Source | Notes |
 |------------------|--------|-------|
-| `id`        | item **title** | trailing `[Nh]` estimate suffix stripped; deduped if needed |
+| `id`        | item **title** | trailing `[Nh]`/`[Nhrs]` estimate suffix stripped; deduped if needed |
 | `prompt`    | item **description** | HTML flattened to plain text |
 | `success`   | constant | always `{ "strategy": "default" }` |
 | `timeoutMs` | constant | `35` minutes → `2100000` ms, unless the user specifies otherwise |
@@ -131,7 +133,7 @@ Options:
 
 - `--timeout-mins N` — override the 35-minute default (the script converts to ms).
 - `--max-attempts N` — override the default of 2.
-- `--keep-estimate` — keep the `[Nh]` suffix in the id instead of stripping it.
+- `--keep-estimate` — keep the `[Nh]`/`[Nhrs]` suffix in the id instead of stripping it.
 - `--output PATH` — write somewhere other than the shared `.agents` queue.
 
 ### 5. Report

package/skills/barrage/scripts/build_queue.py

@@ -8,7 +8,7 @@ produced the input; the queue.json format it writes is documented in ../SKILL.md
 ("Output: the td-barrage queue.json format") so no other repo need be installed.
 
 Field mapping (see ../SKILL.md):
-  - id        <- the item title, with any trailing `[Nh]` estimate suffix stripped
+  - id        <- the item title, with any trailing `[Nh]`/`[Nhrs]` estimate suffix stripped
   - prompt    <- the item description (HTML stripped to plain text)
   - success   <- {"strategy": "default"} on every item
   - timeoutMs <- --timeout-mins * 60 * 1000   (default 35 minutes)
@@ -42,11 +42,14 @@ from queue_paths import resolve_queue_path  # noqa: E402
 # one, whichever copy of the skill is running (see queue_paths.py).
 DEFAULT_QUEUE = resolve_queue_path(SKILL_DIR)
 
-# Matches a trailing estimate suffix like " [2h]" or " [1.5h]" (case-insensitive).
-# Basecamp titles produced by an estimation workflow carry this suffix (the
-# td-augury generator appends "[<hours>h]"); we strip it from the task id so the
-# id reads as a plain title. Titles without a suffix are unaffected.
-ESTIMATE_SUFFIX_RE = re.compile(r"\s*\[\d+(?:\.\d+)?h\]\s*$", re.IGNORECASE)
+# Matches a trailing estimate suffix like " [2h]", " [1.5h]" or " [0.25hrs]"
+# (case-insensitive). Basecamp titles produced by an estimation workflow carry this
+# suffix (the td-augury generator appends "[<hours>h]", but hand-written titles
+# commonly use "hr"/"hrs"/"hour"/"hours"); we strip it from the task id so the id
+# reads as a plain title. Titles without a suffix are unaffected.
+ESTIMATE_SUFFIX_RE = re.compile(
+    r"\s*\[\d+(?:\.\d+)?\s*(?:h|hr|hrs|hour|hours)\]\s*$", re.IGNORECASE
+)
 
 # Block-level tags whose close (or self-close) should become a line break when
 # flattening an HTML card body to plain text.
@@ -228,7 +231,7 @@ def parse_args(argv: list[str] | None = None) -> argparse.Namespace:
     parser.add_argument(
         "--keep-estimate",
         action="store_true",
-        help="Keep the trailing [Nh] estimate suffix in the id (stripped by default).",
+        help="Keep the trailing [Nh]/[Nhrs] estimate suffix in the id (stripped by default).",
     )
     return parser.parse_args(argv)
 

package/skills/barrage/tests/test_build_queue.py

@@ -68,6 +68,19 @@ def test_default_success_timeout_attempts():
     assert tasks[0]["maxAttempts"] == 5
 
 
+def test_estimate_suffix_unit_variants_are_stripped():
+    # Hand-written Basecamp titles spell the unit several ways.
+    for suffix in ("[0.25h]", "[0.25hr]", "[0.25hrs]", "[0.25 hrs]", "[1HOURS]"):
+        item = {
+            "id": 7,
+            "title": f"[KS] ContainedBanner {suffix}",
+            "content": f"[KS] ContainedBanner {suffix}",
+            "description": "Build the banner.",
+        }
+        tasks, _ = build([item])
+        assert tasks[0]["id"] == "[KS] ContainedBanner", suffix
+
+
 def test_keep_estimate_suffix():
     tasks, _ = build([TODO], keep_estimate=True)
     assert tasks[0]["id"] == "Design — Search [2h]"

package/skills/shopify-lint/node_modules/.vite/vitest/da39a3ee5e6b4b0d3255bfef95601890afd80709/results.json

@@ -1 +0,0 @@
-{"version":"4.1.10","results":[[":eslint-plugin-theory/src/rules/guarded-custom-element-define.test.ts",{"duration":0,"failed":true}],[":stylelint-config-theory/src/liquid/mask.test.ts",{"duration":21.486761,"failed":false}],[":theme-check-theory/src/checks/hardcoded-text.test.ts",{"duration":79.642067,"failed":false}],[":stylelint-config-theory/src/integration.test.ts",{"duration":322.234405,"failed":false}],[":theme-check-theory/src/checks/unguarded-nullable-setting.test.ts",{"duration":126.54217099999994,"failed":false}],[":eslint-plugin-theory/src/rules/disconnected-callback-cleanup.test.ts",{"duration":0,"failed":true}],[":eslint-plugin-theory/src/integration.test.ts",{"duration":207.055841,"failed":false}],[":eslint-plugin-theory/src/liquid/mask.test.ts",{"duration":31.769655,"failed":false}],[":stylelint-config-theory/src/config.test.ts",{"duration":13.856021999999996,"failed":false}],[":theme-check-theory/src/checks/unused-section-settings.test.ts",{"duration":63.80623400000002,"failed":false}],[":theme-check-theory/src/checks/unguarded-metaobject.test.ts",{"duration":34.325909000000024,"failed":false}],[":eslint-plugin-theory/src/index.test.ts",{"duration":6.773288999999977,"failed":false}],[":theme-check-theory/src/checks/disallowed-script-or-style-tag.test.ts",{"duration":32.848730000000046,"failed":false}],[":theme-check-theory/src/checks/unguarded-metafield.test.ts",{"duration":37.897699999999986,"failed":false}],[":eslint-plugin-theory/src/rules/no-class-selectors.test.ts",{"duration":0,"failed":true}],[":eslint-plugin-theory/src/rules/no-unapproved-imports.test.ts",{"duration":0,"failed":true}],[":eslint-plugin-theory/src/rules/require-jsdoc.test.ts",{"duration":87.66796799999997,"failed":false}],[":eslint-plugin-theory/src/rules/centralize-selectors.test.ts",{"duration":0,"failed":true}],[":stylelint-config-theory/src/build.test.ts",{"duration":903.9329710000001,"failed":false}],[":theme-check-theory/src/index.test.ts",{"duration":2.380390000000034,"failed":false}],[":eslint-plugin-theory/src/processors/liquid.test.ts",{"duration":6.720345000000009,"failed":false}],[":eslint-plugin-theory/src/build.test.ts",{"duration":965.007413,"failed":false}]]}