siesa-agents 2.1.80 → 2.1.81

package/claude/skills/sa-init-observability/SKILL.md

@@ -73,7 +73,7 @@ Parse the JSON object printed to stdout. The `status` field selects the next bra
 
 Proceed straight to Step 2 with `proposed_slug` and `recommended_dir`. Do **not** ask the user to confirm — the user has already accepted the auto-detection policy by invoking this skill.
 
-**`status: "single-repo"`** — The repo name has no `docs`/`backend`/`frontend` segment, so the git remote already produces an unambiguous `project_id`. Stop and tell the user no `.siesa-project` is needed (mention the `fallback_project_id` value so they know what GCP will see). Do not write anything. If they later say "no, write one anyway with slug X", run Step 2 with that explicit slug and the directory they specify.
+Single-repo projects also return `status: "ready"` (with `convention_match: false`). The proposed slug is the repo name lowercased and sanitized to kebab-case (e.g. `Siesa-Agents` → `siesa-agents`) and `recommended_dir` is the git root itself. Proceed to Step 2 the same way — no separate branch.
 
 **`status: "already-configured"`** — A `.siesa-project` exists above the current git root. Stop and report `existing_siesa_project.path` and `existing_siesa_project.value` to the user. Do not overwrite implicitly. If the user replies that they want it changed, run `--write` with `--force`, the new slug, and the same directory.
 
@@ -95,7 +95,9 @@ Parse the JSON response:
 - **`status: "exists"`** — A file is already there with a different value (the detect step missed it because the user pointed at a non-default dir, or a race). Show `current_value` vs `requested_slug` and ask the user whether to overwrite. If yes, re-run with `--force`.
 - **`status: "error"`** — Surface the error message verbatim and stop.
 
-For the `single-repo`, `already-configured` and `no-remote` branches where the user explicitly opts into writing, gather slug + directory inline and run the same `--write` command.
+For the `already-configured` and `no-remote` branches where the user explicitly opts into writing, gather slug + directory inline and run the same `--write` command.
+
+The file body written by `--write` is always `project_id=<slug>\n` (key=value form). `sa-emit.js` accepts both this format and a legacy bare-slug line for backward compatibility, so existing `.siesa-project` files keep working.
 
 ### Step 3 — Verify (optional)
 
@@ -129,7 +131,7 @@ business-<module>-<role>-<rest>
 
 The logical slug collapses `<role>` away: `business-pos-backend-pdv` → `pos-pdv`.
 
-For repo names that do not contain `docs`/`backend`/`frontend`, the script returns `single-repo` and no `.siesa-project` is needed.
+For repo names that do not contain `docs`/`backend`/`frontend`, the script still returns `status: "ready"` but with `convention_match: false`. The slug is derived from the repo name (lowercased, sanitized to kebab-case) and the file is written at the git root of that single repo.
 
 ## Examples
 
@@ -157,11 +159,11 @@ Model: <runs sa-init-env.js — status "created">
 
 ```
 User: Set up .siesa-project for Siesa-Agents.
-Model: <runs --detect from Siesa-Agents/>
-       Repo name "Siesa-Agents" has no docs/backend/frontend segment, so the git remote already produces an unambiguous project_id ("SiesaTeams/Siesa-Agents"). No .siesa-project is needed.
-       Want to set a custom slug anyway?
-User: no, thanks
-Model: Nothing written. Existing fallback (SiesaTeams/Siesa-Agents) will continue to be reported.
+Model: <runs sa-init-env.js — silent>
+       <runs --detect from Siesa-Agents/>
+       <reads proposed_slug="siesa-agents", recommended_dir=C:\…\Siesa-Agents, convention_match=false>
+       <runs --write --slug siesa-agents --dir C:\…\Siesa-Agents>
+       Wrote C:\…\Siesa-Agents\.siesa-project (project_id=siesa-agents). sa-emit.js will now report project_id="siesa-agents" instead of "SiesaTeams/Siesa-Agents".
 ```
 
 **Example 3 — Already configured**

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "siesa-agents",
-  "version": "2.1.80",
+  "version": "2.1.81",
   "description": "Paquete para instalar y configurar agentes SIESA en tu proyecto",
   "main": "index.js",
   "bin": {

package/siesa-agents/observability/scripts/sa-emit.js

@@ -134,17 +134,38 @@ const stateDir = path.join(os.homedir(), '.claude', 'observability')
 const epicIdMatch = story.match(/^(\d+)-/)
 const epicId      = epicIdMatch ? epicIdMatch[1] : 'unknown'
 
+function parseSiesaProject(content) {
+  // Preferred format: a single `project_id=<slug>` line. For backward compatibility a bare
+  // slug on its own line is also accepted. `#` introduces a comment. Returns the resolved
+  // slug, or null if nothing usable was found.
+  if (!content) return null
+  let firstBare = null
+  for (const raw of content.split(/\r?\n/)) {
+    const line = raw.trim()
+    if (!line || line.startsWith('#')) continue
+    const eq = line.indexOf('=')
+    if (eq !== -1) {
+      const key = line.slice(0, eq).trim()
+      const val = line.slice(eq + 1).trim()
+      if (key === 'project_id' && val) return val
+      continue
+    }
+    if (firstBare === null) firstBare = line
+  }
+  return firstBare
+}
+
 function findSiesaProject(startDir) {
-  // Walk up from startDir looking for a `.siesa-project` file. Returns its first non-empty
-  // line trimmed, or null. Lets a parent dir override the git-remote-derived project_id —
+  // Walk up from startDir looking for a `.siesa-project` file. Returns the project_id slug
+  // it declares, or null. Lets a parent dir override the git-remote-derived project_id —
   // useful when several sub-repos (docs/backend/frontend) belong to the same logical project.
   let cur = path.resolve(startDir)
   for (let i = 0; i < 40; i++) {
     const candidate = path.join(cur, '.siesa-project')
     if (fs.existsSync(candidate)) {
       try {
-        const first = fs.readFileSync(candidate, 'utf8').split(/\r?\n/)[0].trim()
-        if (first) return first
+        const slug = parseSiesaProject(fs.readFileSync(candidate, 'utf8'))
+        if (slug) return slug
       } catch (_) {}
     }
     const parent = path.dirname(cur)

package/siesa-agents/observability/scripts/sa-init-project.js

@@ -63,6 +63,27 @@ function tryExec(cmd, opts) {
   }
 }
 
+// Parse a `.siesa-project` file body. Preferred format is `project_id=<slug>` (one key=value
+// line, optional `#` comments). For backward compatibility a bare slug on its own line is
+// also accepted. Returns the resolved slug, or null if nothing usable was found.
+function parseSiesaProject(content) {
+  if (!content) return null
+  let firstBare = null
+  for (const raw of content.split(/\r?\n/)) {
+    const line = raw.trim()
+    if (!line || line.startsWith('#')) continue
+    const eq = line.indexOf('=')
+    if (eq !== -1) {
+      const key = line.slice(0, eq).trim()
+      const val = line.slice(eq + 1).trim()
+      if (key === 'project_id' && val) return val
+      continue
+    }
+    if (firstBare === null) firstBare = line
+  }
+  return firstBare
+}
+
 // Walk up from `startDir` looking for an existing `.siesa-project`. Returns
 // { path, value } of the first match found, or null.
 function findExistingSiesaProject(startDir) {
@@ -71,8 +92,8 @@ function findExistingSiesaProject(startDir) {
     const candidate = path.join(cur, '.siesa-project')
     if (fs.existsSync(candidate)) {
       try {
-        const first = fs.readFileSync(candidate, 'utf8').split(/\r?\n/)[0].trim()
-        return { path: candidate, value: first }
+        const value = parseSiesaProject(fs.readFileSync(candidate, 'utf8'))
+        return { path: candidate, value }
       } catch (_) {
         return { path: candidate, value: null }
       }
@@ -186,15 +207,38 @@ function runDetect() {
   const derived = deriveLogicalSlug(repoName)
 
   if (!derived.matched) {
+    // Single-repo project: still write a `.siesa-project` so the project_id is explicit
+    // and self-documented. The slug is derived from the repo name, sanitized to kebab-case,
+    // and the file lands at the git root of the current project.
+    const singleRepoSlug = (repoName || '')
+      .toLowerCase()
+      .replace(/[^a-z0-9]+/g, '-')
+      .replace(/^-+|-+$/g, '')
+    if (!singleRepoSlug || !/^[a-z0-9][a-z0-9-]*[a-z0-9]$/.test(singleRepoSlug)) {
+      emit({
+        status: 'no-remote',
+        message: 'Could not derive a valid kebab-case slug from the repo name. Pass --slug explicitly to write a .siesa-project.',
+        git_root: gitRoot,
+        remote_url: remoteUrl,
+        remote_repo: repoName,
+      })
+      process.exit(2)
+    }
     emit({
-      status: 'single-repo',
-      message: 'This repo name does not match the multi-repo convention (no docs|backend|frontend segment). sa-emit.js will report project_id based on the git remote, which is already unambiguous for single-repo projects. No .siesa-project is required.',
+      status: 'ready',
+      message: 'Single-repo project detected. The .siesa-project file will be written at the git root with a slug derived from the repo name.',
       git_root: gitRoot,
       remote_url: remoteUrl,
       remote_repo: repoName,
+      convention_match: false,
+      proposed_slug: singleRepoSlug,
+      convention_breakdown: null,
+      parent_candidates: [{ path: gitRoot, label: 'git root', levelsUp: 0 }],
+      recommended_dir: gitRoot,
       fallback_project_id: remoteParsed ? `${remoteParsed.org}/${remoteParsed.repo}` : null,
+      next_step_example: `node sa-init-project.js --write --slug "${singleRepoSlug}" --dir "${gitRoot}"`,
     })
-    process.exit(2)
+    process.exit(0)
   }
 
   const candidates = listParentCandidates(gitRoot)
@@ -256,7 +300,7 @@ function runWrite(args) {
   const target = path.join(resolvedDir, '.siesa-project')
   if (fs.existsSync(target) && !force) {
     let current = null
-    try { current = fs.readFileSync(target, 'utf8').split(/\r?\n/)[0].trim() } catch (_) {}
+    try { current = parseSiesaProject(fs.readFileSync(target, 'utf8')) } catch (_) {}
     emit({
       status: 'exists',
       message: '.siesa-project already exists at this location. Pass --force to overwrite.',
@@ -268,7 +312,7 @@ function runWrite(args) {
   }
 
   try {
-    fs.writeFileSync(target, normalizedSlug + '\n', { encoding: 'utf8' })
+    fs.writeFileSync(target, `project_id=${normalizedSlug}\n`, { encoding: 'utf8' })
   } catch (err) {
     emit({ status: 'error', message: `Failed to write ${target}: ${err.message}` })
     process.exit(3)