@ggdna/dna-gg 1.3.0 → 1.4.1

package/README.md

@@ -39,7 +39,7 @@ All skills of this layer are prefixed with `gg-`:
 - `/gg-commit` — proposes a message and commits through `gg do commit`
 - `/gg-push` — pushes the ticket branches
 - `/gg-review` — checks the scope of every repo, writes the blog posts
-  and opens the pull requests
+  and opens the pull requests, printing their URLs
 - `/gg-publish` — releases the ticket
 - `/gg-cleanup` — removes what the ticket left behind
 
@@ -52,10 +52,12 @@ All skills of this layer are prefixed with `gg-`:
   prompts wait in `~/.claude/pm-prompts/sessions/` and are committed to the
   branch `prompts` of that repo when a session starts or ends, under
   `doc/prompts/<first name>/<yyyy-mm>/<yyyy-mm-dd>_<repo>_<session>.md`.
-  The first name comes from the `Git` line of `doc/team/people.md` there,
-  an unknown email goes to `doc/prompts/_unknown/`. Only the prompts are
-  kept, never the answers; tokens and passwords are replaced by
-  `[redacted]`. No working tree is touched: the commit is built in a bare
+  The first name comes from `doc/team/people.md` there, found by the git
+  user name: a first name equal to it or to its first word, or a `Git`
+  line listing it. An unknown name goes to `doc/prompts/_unknown/`. No
+  e-mail address is read or written; the commits carry the neutral author
+  `pm-prompts`. Only the prompts are kept, never the answers; tokens and
+  passwords are replaced by `[redacted]`. No working tree is touched: the commit is built in a bare
   cache below `~/.claude/pm-prompts/repos/`. Needs `node` and `git`; a
   failure never stops a prompt and ends up in
   `~/.claude/pm-prompts/sync.log`.

package/dna/_generated.json

@@ -39,7 +39,7 @@
       "ecosystem": null,
       "resolvedVersion": null,
       "via": null,
-      "hash": "0x7e5e4bbb754c4f13"
+      "hash": "0x958d0143021b2328"
     }
   ],
   "claude": {
@@ -59,7 +59,7 @@
     },
     {
       "path": ".claude/hooks/pm-prompts.mjs",
-      "hash": "0x988aed96a8073727"
+      "hash": "0xf4266eea17b3b9c1"
     },
     {
       "path": ".claude/settings.json",
@@ -83,11 +83,11 @@
     },
     {
       "path": ".claude/skills/gg-review/SKILL.md",
-      "hash": "0x55b2f0deb18d70d4"
+      "hash": "0x3a4506ccd5c8e1be"
     },
     {
       "path": ".claude/skills/gg-ticket/SKILL.md",
-      "hash": "0x0532d8ac37060521"
+      "hash": "0xb1095b91b3531897"
     },
     {
       "path": ".claude/skills/gg/SKILL.md",
@@ -107,7 +107,7 @@
     },
     {
       "path": "doc/guides/develop-guide.md",
-      "hash": "0x39718de56fdb48f0"
+      "hash": "0xed4328d138fe76ef"
     },
     {
       "path": "doc/guides/develop-without-ai-guide.md",
@@ -115,7 +115,7 @@
     },
     {
       "path": "doc/guides/for-ai/ai-dev-guide.md",
-      "hash": "0xb6eecdb4c7a1ebe8"
+      "hash": "0xe8182903f11a1766"
     },
     {
       "path": "doc/guides/index-guide.md",

package/dna/doc/guides/develop-guide.md

@@ -47,6 +47,8 @@ Use the following skills in Claude
 When the repo belongs to a project management repo, every prompt you write
 to Claude Code is kept there, under `doc/prompts/<your first name>/` on the
 branch `prompts`. Only the prompts, not the answers. Tokens and passwords
-are replaced by `[redacted]`. Your first name comes from the `Git` line of
-`doc/team/people.md` in the project management repo; add your git email
-there, or your prompts land in `doc/prompts/_unknown/`.
+are replaced by `[redacted]`. Your first name comes from
+`doc/team/people.md` in the project management repo: the person whose
+first name is your git user name or its first word, or whose `Git` line
+lists your git user name. Otherwise your prompts land in
+`doc/prompts/_unknown/`. Your e-mail address never goes there.

package/dna/doc/guides/for-ai/ai-dev-guide.md

@@ -8,31 +8,43 @@ found in the LICENSE file in the root of this package.
 
 # Development guide for AIs
 
-## Ask the developer for the ticket infos
+## Ask the developer what the ticket should do
 
-Ask for these one after the other and give the developer a single input
-field per answer — one question per field, never all of them in one
-question and never as a plain prose question:
+Ask a single question: what should the ticket do? One free-text answer is
+enough — never ask for a ticket id, a title or a description separately.
 
-- Ask for the ticket ID
-- Ask for the ticket title
-- Ask for the ticket description
-
-## Replace in this document
+## Create a ticket
 
-- Replace `~/dev/` by the workspace directory from the memory
-- Ask for the workspace directory if none is known
-- Replace `DnaJiraPrefix-145` by the ticket ID you asked for
-- Replace `Fix issue abc` by the ticket title you asked for
+Derive from the answer, without asking anything else:
 
-## Create a ticket
+- **id** — a short, descriptive slug in `snake_case`, 2-5 plain ASCII words
+  (transliterate non-ASCII, drop punctuation and emoji), e.g.
+  `fix_login_crash`. Never a sequential number: several tickets are usually
+  worked on in parallel, and a counter collides between them. List the
+  existing ticket folders of the workspace root and, if the slug is taken,
+  append `_2`, `_3`, … until it is free. If `gg do create ticket` still
+  rejects it as taken (e.g. another agent just created the same slug),
+  bump the suffix and retry.
+- **message** — one imperative line to one short sentence describing the
+  change, for `-m`.
 
 ```bash
 cd ~/dev/ # workspace
-gg do create ticket DnaJiraPrefix-145 -m"Fix issue abc"
-cd tickets/DnaJiraPrefix-145
+gg do create ticket fix_login_crash -m"Fix issue abc"
+cd fix_login_crash
 ```
 
+Tell the developer the id and the message right after creating the ticket —
+both can be changed any time, e.g. by renaming the folder before anything
+else is added to it, or by editing `ticket.json`.
+
+## Replace in this document
+
+- Replace `~/dev/` by the workspace directory from the memory
+- Ask for the workspace directory if none is known
+- Replace `fix_login_crash` by the ticket id you derived
+- Replace `Fix issue abc` by the ticket message you derived
+
 ## Choose the project management repo — or work without one
 
 A ticket may belong to a project management repo. It holds the plans,
@@ -62,8 +74,9 @@ repositories. The ticket then carries no plan of its own.
 
 ## Add git repositories
 
-Ask the user how the repos are chosen, as a single question with two
-options: automatically or manually.
+Ask the user how the repos are chosen, as a single question with the
+options below. Offer **only the project management repo** only when one was
+added above — without one there is nothing to be "only" about.
 
 Manually: ask in a second single input field which repos the ticket needs,
 e.g. `repo1 repo2`. Check each name against the repos in .ocean and ask
@@ -80,8 +93,10 @@ Also explain to the user what you roughly want to change in which repo to
 implement the ticket and let them confirm that the corresponding repos are
 added to the ticket.
 
-If the ticket is only planned (see below), the project management repo may
-be the only repo of the ticket.
+Only the project management repo: no code repo at all; the ticket stays
+pure planning, and the project management repo added above is everything
+it carries. If the ticket is only planned (see below), this is the way to
+go — the project management repo may be the only repo of the ticket.
 
 After the user's confirmation, add the repos to the ticket:
 
@@ -321,8 +336,8 @@ gg do review
 ```
 
 gg creates pull requests for each repo and prints the URLs to the
-terminal. Print every URL to the user, one line per repo, and open each
-pull request without auto-merge in the browser.
+terminal. Print every URL to the user, one line per repo. Do not open the
+pull requests in the browser — leave that to the user.
 
 Afterwards load the review-light skill and execute it.
 

package/dna/dot-claude/hooks/pm-prompts.mjs

@@ -238,7 +238,7 @@ export function onPrompt(input) {
       started: new Date().toISOString(),
       folder: path.basename(projectDir),
       repos,
-      email: git(['-C', projectDir, 'config', 'user.email']).out,
+      name: git(['-C', projectDir, 'config', 'user.name']).out,
       targets,
     });
   }
@@ -299,28 +299,35 @@ export function slug(name) {
     .replace(/^-+|-+$/g, '');
 }
 
-/** The first name in [peopleMd] whose `Git` line lists [email]. */
-export function personOf(email, peopleMd) {
-  if (!email) return '';
+/**
+ * The first name in [peopleMd] for the git user name [gitName]: the person
+ * whose `Git` line lists it, else the person whose first name is the git
+ * name or its first word. E-mail addresses play no part.
+ */
+export function personOf(gitName, peopleMd) {
+  const wanted = (gitName ?? '').trim().toLowerCase();
+  if (!wanted) return '';
+  const byFirstName = new Map();
   let name = '';
   for (const line of peopleMd.split(/\r?\n/)) {
     const heading = line.match(/^##\s+(.+?)\s*$/);
     if (heading) {
       name = heading[1];
+      byFirstName.set(name.toLowerCase(), name);
       continue;
     }
-    const emails = line.match(/^\s*-\s*Git:\s*(.+)$/i);
+    const names = line.match(/^\s*-\s*Git:\s*(.+)$/i);
     if (
       name &&
-      emails &&
-      emails[1]
-        .split(/[,;\s]+/)
-        .some((e) => e.toLowerCase() === email.toLowerCase())
+      names &&
+      names[1].split(/[,;]/).some((n) => n.trim().toLowerCase() === wanted)
     ) {
       return name;
     }
   }
-  return '';
+  return (
+    byFirstName.get(wanted) ?? byFirstName.get(wanted.split(/\s+/)[0]) ?? ''
+  );
 }
 
 /** Where the prompts of a session go in the project management repo. */
@@ -336,7 +343,7 @@ export function renderHead(meta, person) {
   const lines = [
     `# ${localDate(meta.started)} · ${meta.folder} · ${String(meta.session).slice(0, 8)}`,
     '',
-    `- Person: ${person || meta.email || 'unknown'}`,
+    `- Person: ${person || meta.name || 'unknown'}`,
   ];
   const repos = meta.repos ?? [];
   if (repos.length === 1 && repos[0] === meta.folder) {
@@ -399,16 +406,15 @@ function fetchBranch(dir) {
   return git(['-C', dir, 'rev-parse', ref]).out;
 }
 
-function identityEnv(dir) {
-  const env = { ...gitEnv };
-  if (!git(['-C', dir, 'config', 'user.name']).out) {
-    env.GIT_AUTHOR_NAME = env.GIT_COMMITTER_NAME = 'pm-prompts';
-  }
-  if (!git(['-C', dir, 'config', 'user.email']).out) {
-    env.GIT_AUTHOR_EMAIL = env.GIT_COMMITTER_EMAIL = 'pm-prompts@localhost';
-  }
-  return env;
-}
+// The prompts are committed under a neutral identity, so the e-mail address
+// of the developer never reaches the project management repo.
+const identityEnv = {
+  ...gitEnv,
+  GIT_AUTHOR_NAME: 'pm-prompts',
+  GIT_COMMITTER_NAME: 'pm-prompts',
+  GIT_AUTHOR_EMAIL: 'pm-prompts@localhost',
+  GIT_COMMITTER_EMAIL: 'pm-prompts@localhost',
+};
 
 // Commits the not yet pushed prompts of [sessions] to the prompts branch of
 // [url]. Returns how many prompts of each session the branch now holds.
@@ -421,7 +427,7 @@ function pushTarget(url, sessions) {
       os.tmpdir(),
       `pm-prompts-${process.pid}-${Date.now()}.index`,
     );
-    const env = { ...identityEnv(dir), GIT_INDEX_FILE: index };
+    const env = { ...identityEnv, GIT_INDEX_FILE: index };
     try {
       const read = base
         ? git(['-C', dir, 'read-tree', base], { env })
@@ -433,8 +439,8 @@ function pushTarget(url, sessions) {
         const fresh = session.prompts.slice(pushed);
         done.set(file, session.prompts.length);
         if (!fresh.length) continue;
-        const person = personOf(session.meta.email, people);
-        names.add(person || session.meta.email || 'unknown');
+        const person = personOf(session.meta.name, people);
+        names.add(person || session.meta.name || 'unknown');
         const target = promptsPath(session.meta, person);
         const existing = base
           ? git(['-C', dir, 'cat-file', '-p', `${base}:${target}`])

package/dna/dot-claude/skills/gg-review/SKILL.md

@@ -1,6 +1,6 @@
 ---
 name: gg-review
-description: Writes the blog posts of the ticket, opens the pull requests with `gg do review` and reviews the changes. Use when the user says "/gg-review" or asks to review the current ticket.
+description: Writes the blog posts of the ticket, opens the pull requests with `gg do review` and prints their URLs, and reviews the changes. Use when the user says "/gg-review" or asks to review the current ticket.
 ---
 
 # Review
@@ -57,21 +57,16 @@ output, so pass the URLs on:
 - Print every URL to the user, one line per repo: `<repo>: <url>`. Name
   each repo that gg reports as `No pull request for <repo>`, with the
   reason
-- Open every pull request without auto-merge in the browser. Ask the
-  provider whether auto-merge is set:
-  - GitHub: `gh pr view <url> --json autoMergeRequest` — `null` means off
-  - Azure DevOps: `az repos pr show --id <id> --query autoCompleteSetBy`
-    — empty means off
-- Open the URL with `start "" "<url>"` on Windows, `open "<url>"` on
-  macOS and `xdg-open "<url>"` on Linux
+- Do not open the pull requests in the browser — print the URLs and leave
+  opening them to the user.
 
 ## 4. Review the changes
 
 Load the `/review-light` skill and execute it.
 
 A fix goes through `/gg-commit` and `/gg-push`, then run `gg do review`
-again. Print and open its pull request URLs as in step 3. Update the blog
-post of the repo when the fix changes what it describes.
+again. Print its pull request URLs as in step 3. Update the blog post of
+the repo when the fix changes what it describes.
 
 ## Important
 
@@ -80,3 +75,4 @@ post of the repo when the fix changes what it describes.
   the project management repo.
 - Never start `gg do review` without the user's confirmation.
 - Always print the pull request URLs after `gg do review`.
+- Never open a pull request in the browser on your own.

package/dna/dot-claude/skills/gg-ticket/SKILL.md

@@ -8,24 +8,36 @@ description: Creates a gg ticket and adds the repos it needs. Use when the user
 All `gg do` commands run inside the workspace. Ask for the workspace
 directory if none is known.
 
-## 1. Ask for the ticket data
+## 1. Ask what the ticket should do
 
-Ask for these three, one after the other, and give the user a single input
-field per answer — one question per field, never all three in one question
-and never as a plain prose question:
-
-- Ticket ID, e.g. `DnaJiraPrefix-145`
-- Title, one imperative line, e.g. `Fix issue abc`
-- Description, one or two sentences on what the ticket changes
+Ask a single question: what should the ticket do? One free-text answer is
+enough — never ask for a ticket id, a title or a description separately.
 
 ## 2. Create the ticket
 
+Derive from the answer, without asking anything else:
+
+- **id** — a short, descriptive slug in `snake_case`, 2-5 plain ASCII words
+  (transliterate non-ASCII, drop punctuation and emoji), e.g.
+  `fix_login_crash`. Never a sequential number: several tickets are usually
+  worked on in parallel, and a counter collides between them. List the
+  existing ticket folders of the workspace root and, if the slug is taken,
+  append `_2`, `_3`, … until it is free. If `gg do create ticket` still
+  rejects it as taken (e.g. another agent just created the same slug),
+  bump the suffix and retry.
+- **message** — one imperative line to one short sentence describing the
+  change, for `-m`.
+
 ```bash
 cd ~/dev/ # workspace
-gg do create ticket DnaJiraPrefix-145 -m"Fix issue abc"
-cd tickets/DnaJiraPrefix-145
+gg do create ticket fix_login_crash -m"Fix issue abc"
+cd fix_login_crash
 ```
 
+Tell the user the id and the message right after creating the ticket — both
+can be changed any time, e.g. by renaming the folder before anything else is
+added to it, or by editing `ticket.json`.
+
 ## 3. Choose the project management repo — or work without one
 
 A ticket may belong to a project management repo: plans, decisions and blog
@@ -52,8 +64,9 @@ Call it the "project management repo" whenever you write to the user, never
 
 ## 4. Choose the code repos
 
-Ask the user how the code repos are chosen, as a single question with two
-options:
+Ask the user how the code repos are chosen, as a single question with the
+options below. Offer **only the project management repo** only when one was
+added in step 3 — without one there is nothing to be "only" about.
 
 - **automatically** — Claude finds the repos. Read the `index.jsonc` of
   the candidate repos in `.ocean`. Tell the user which repos the ticket
@@ -65,6 +78,9 @@ options:
   field which repos the ticket needs, e.g. `repo1 repo2`. Check each name
   against the repos in `.ocean` and ask again for names that do not exist
   there. The entered list is the confirmation.
+- **only the project management repo** — no code repo at all; the ticket
+  stays pure planning, and the project management repo added in step 3 is
+  everything it carries.
 
 A ticket that is only planned may need no code repo at all.
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@ggdna/dna-gg",
-  "version": "1.3.0",
+  "version": "1.4.1",
   "description": "DNA layer: gg CLI workflow guides, repo setup scripts and CI check",
   "keywords": [
     "dna",