autark-cli 0.5.5 → 0.5.7

package/autark.mjs

@@ -517,7 +517,9 @@ async function refreshRuntimeFiles() {
 }
 
 async function refreshSkills() {
-  const res = await spawnSync('pnpm', ['dlx', '--silent', 'skills', 'add', 'kiluazen/kstack', '-s', SKILL_NAMES.join(','), '-y', '-g'])
+  // skills CLI switched from `-s name,name` to positional `name name`.
+  // The old comma-separated form silently matches nothing.
+  const res = await spawnSync('pnpm', ['dlx', '--silent', 'skills', 'add', 'kiluazen/kstack', ...SKILL_NAMES, '-y', '-g'])
   if (res.status !== 0) throw new Error(`kstack skills install failed`)
 }
 
@@ -774,15 +776,31 @@ async function hypothesisPause(rest, status) {
 async function feedbackRecord(rest) {
   const opts = parseArgs(rest)
   const ref = opts.hypothesis || opts._[0]
+  const text = readValue(required(opts.text || opts.message || opts.body, '--text "<feedback>"'))
+
+  // --product-id (or a bare product slug with no /Hxx) → product-level feedback.
+  const productId = opts['product-id'] || opts.product_id
+  const isProductSlug = ref && !opts['hypothesis-id'] && !opts.hypothesis_id && !/\/H\d{2}$/.test(ref)
+  if (productId || isProductSlug) {
+    let pid = productId
+    if (!pid) {
+      const ctx = await api('GET', `/v1/context/${encodeURIComponent(ref)}`)
+      pid = ctx.product?.id
+    }
+    if (!pid) throw new Error('product not found')
+    const result = await api('POST', `/v1/products/${encodeURIComponent(pid)}/feedback`, { text })
+    console.log(result.id)
+    return
+  }
+
   let hypId = opts['hypothesis-id'] || opts.hypothesis_id
   if (!hypId) {
-    if (!ref) throw new Error('pass product/Hxx or --hypothesis-id')
+    if (!ref) throw new Error('pass product, product/Hxx, --product-id, or --hypothesis-id')
     const [productSlug, code] = splitHypothesisRef(ref)
     const ctx = await api('GET', `/v1/context/${encodeURIComponent(productSlug)}/${encodeURIComponent(code)}`)
     hypId = ctx.hypothesis?.id
   }
   if (!hypId) throw new Error('hypothesis not found')
-  const text = readValue(required(opts.text || opts.message || opts.body, '--text "<feedback>"'))
   const body = { text }
   if (opts['action-id'] || opts.action_id) body.action_id = opts['action-id'] || opts.action_id
   const result = await api('POST', `/v1/hypotheses/${encodeURIComponent(hypId)}/feedback`, body)
@@ -797,7 +815,8 @@ async function feedbackDelete(rest) {
 }
 
 function feedbackUsage() {
-  console.log(`autark feedback record <product/Hxx> --text "<nudge>" [--action-id <id>]
+  console.log(`autark feedback record <product/Hxx> --text "<nudge>" [--action-id <id>]   per-hypothesis nudge
+autark feedback record <product>     --text "<nudge>"                     product-level note (any hypothesis)
 autark feedback delete <feedback-id>`)
 }
 
@@ -884,7 +903,16 @@ async function mailLint(rest) {
   const compoundQ = /\b(are|is|do|does|did|will|would|can|could|should|have|has)\b[^?]{0,200}\bor\b[^?]{0,200}\?/i
   if (compoundQ.test(body)) violations.push({ rule: 'compound-question', detail: 'a/b question with "or"', why: WHY['compound-question'] })
   const sigSplit = body.split(/\n\s*\n(?:best|thanks|cheers|—|-)[\s,]*\n/i)[0] || body
-  const len = sigSplit.trim().length
+  // Measure VISIBLE prose length, not raw markup. An <a href>/[text](url) signature
+  // anchor or any HTML wrapper must not eat into the 400-char content budget — only
+  // what the recipient actually reads counts.
+  const visibleText = sigSplit
+    .replace(/<a\b[^>]*>(.*?)<\/a>/gis, '$1')           // <a ...>Kushal</a> -> Kushal
+    .replace(/\[([^\]]+)\]\(\s*https?:\/\/[^)]+\)/g, '$1') // [Kushal](url) -> Kushal
+    .replace(/<[^>]+>/g, '')                             // drop remaining html tags
+    .replace(/\s+/g, ' ')                               // collapse whitespace
+    .trim()
+  const len = visibleText.length
   if (len > 400) violations.push({ rule: 'too-long', detail: `${len} chars`, why: WHY['too-long'] })
   const hasMdAnchor = /\[[^\]]+\]\(https?:\/\/[^)]+\)/.test(body)
   const hasHtmlAnchor = /<a\b[^>]*href\s*=\s*["']https?:\/\/[^"']+["'][^>]*>[^<]+<\/a>/i.test(body)
@@ -1186,6 +1214,14 @@ function printProductContext(r) {
   kv('product.url', r.product.url)
   kv('product.visibility', r.product.visibility)
   kv('product.hypothesis_count', (r.hypotheses || []).length)
+  // Product-level operator feedback (newest first). Append-only + timestamped,
+  // so each entry is a distinct nudge — re-read these every run.
+  kv('product.feedback_count', (r.feedback || []).length)
+  for (const f of (r.feedback || [])) {
+    const shortF = String(f.id || '').slice(0, 8)
+    kv(`product.feedback.${shortF}.created_at`, f.created_at)
+    kv(`product.feedback.${shortF}.text`, f.text)
+  }
   for (const h of (r.hypotheses || [])) {
     kv(`hypothesis.${h.code}.id`, h.id)
     kv(`hypothesis.${h.code}.title`, h.title)
@@ -1207,6 +1243,13 @@ function printProductContext(r) {
   if (r.product.tagline) console.log(`> ${r.product.tagline}\n`)
   console.log(`## Brief\n`)
   console.log(r.product.brief?.trim() || '(no brief set — owner can add one at https://autark.sh)')
+  if ((r.feedback || []).length > 0) {
+    console.log(`\n## Operator feedback (${r.feedback.length}) — product-level, newest first\n`)
+    console.log(`These are general operator nudges, not tied to one hypothesis. Treat the most recent as the freshest signal and let them re-shape your next move.\n`)
+    for (const f of r.feedback) {
+      console.log(`- (${f.created_at}) ${f.text}`)
+    }
+  }
   if (!r.hypotheses?.length) {
     console.log(`\n## Hypotheses\n\n(none yet — create one with: autark hypothesis create --product-id <product_id> --md @hyp.md)`)
   } else {
@@ -1469,7 +1512,7 @@ function usage() {
   run start|finish           start / finish a run
   log action                 record one outreach touch
   action escalate            flag an action for human attention
-  feedback record|delete     leave a free-text nudge on a hypothesis
+  feedback record|delete     leave a free-text nudge on a hypothesis or product
   context [<slug>|...]       pull product or hypothesis context
 
   mail setup|send|reply|...  send/read mail via your AgentMail inbox

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "autark-cli",
-  "version": "0.5.5",
+  "version": "0.5.7",
   "description": "CLI for autark \u2014 hypothesis-driven product runbooks. Track products, hypotheses, runs, and actions from the terminal.",
   "type": "module",
   "license": "MIT",