@devflow-one/cli 0.1.7 → 0.1.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@devflow-one/cli",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "description": "Terminal CLI wrapping the local DevFlow gateway so shell scripts can trigger repo actions.",
   "keywords": [
     "devflow",

package/src/cli.js

@@ -77,6 +77,15 @@ function printResult(result, json, stdout) {
   }
 }
 
+/** Echo a result's `warnings` to stderr so they stay visible next to --json stdout. */
+function printWarnings(result, stderr) {
+  const warnings = Array.isArray(result?.warnings) ? result.warnings : [];
+
+  for (const warning of warnings) {
+    stderr(`warning: ${warning}`);
+  }
+}
+
 /** Print an error, honoring --json, to stderr. */
 function printError(err, json, stderr) {
   const code = err instanceof GatewayError ? err.code : err.name;
@@ -217,6 +226,7 @@ export async function run(argv, io = {}) {
     const result = await request(config, "POST", `/actions/${action}`, body);
 
     printResult(result, Boolean(flags.json), stdout);
+    printWarnings(result, stderr);
 
     return 0;
   } catch (err) {

package/src/commands.js

@@ -143,7 +143,10 @@ const COMMANDS = {
     action: "commitByTemplate",
     usage:
       'commit template <worktreePath> [--values \'{"key":"val"}\'] [--no-push] [--issue KEY] [--project KEY]' +
-      " (--values keys must match the {key} tokens of the project's configured commit template)",
+      " (--values keys fill the project's commit template, default" +
+      ' "{summary}\\n\\n{changesList}\\n\\n{issueSlug}": summary, changesList (string array,' +
+      " rendered as bullets), issueSlug, prNumber; missing keys are left empty and reported" +
+      " as warnings on stderr)",
     buildArgs: ([worktreePath], flags) => {
       if (!worktreePath) throw new UsageError("commit template requires <worktreePath>.");
 

package/src/context.js

@@ -5,7 +5,7 @@
 // The resolver itself (contextResolver.js) is generated from shared/ and must not
 // be edited here; everything CLI-specific lives in this file.
 
-import { resolveContext } from "./contextResolver.js";
+import { issueKeyToProjectKey, resolveContext } from "./contextResolver.js";
 import { createGitRunner } from "./gitFacts.js";
 import { GatewayError } from "./client.js";
 import { UsageError } from "./commands.js";
@@ -84,13 +84,7 @@ export function resolveCliContext(
 export function projectKeyFrom(context) {
   if (context.projectKey) return context.projectKey.toUpperCase();
 
-  if (context.issueKey) {
-    const lastDash = context.issueKey.lastIndexOf("-");
-
-    return (lastDash === -1 ? context.issueKey : context.issueKey.slice(0, lastDash)).toUpperCase();
-  }
-
-  return undefined;
+  return issueKeyToProjectKey(context.issueKey);
 }
 
 /** Describe one ambiguity candidate as `PROJECT/repo (profile)`. */

package/src/contextResolver.js

@@ -56,11 +56,11 @@ function clean(value) {
 
 /**
  * The project a key or issue key belongs to: split on the LAST `-` and uppercase
- * (`abc-123` → `ABC`). Mirrors `issueKeyToProjectKey` in the daemon's routing.js.
- * @param {string | undefined} value
- * @returns {string | undefined}
+ * (`abc-123` → `ABC`). A value with no `-` is already a project key.
+ * @param {unknown} value
+ * @returns {string | undefined} `undefined` for a non-string or blank value
  */
-function projectOf(value) {
+export function issueKeyToProjectKey(value) {
   const trimmed = clean(value);
 
   if (!trimmed) return undefined;
@@ -227,16 +227,10 @@ export async function resolveContext({
     // so the reported precedence step reflects the real reason, not a guess.
     const origin = context.matchedBy === "remote" ? "gitRemote" : "localPath";
 
-    // The daemon answered "this CHECKOUT belongs to project P". Its repo, profile,
-    // liveness and issue key all describe P — they are only meaningful together.
-    // If the caller already pinned a different project, adopting them field-by-field
-    // would aim the action at P's repository under the caller's key: silent
-    // mis-targeting, and worse than a failure because nothing looks wrong. So the
-    // whole group is discarded on disagreement. Note this compares the KEYS, not
-    // merely where they came from — a caller who names the project the checkout
-    // already matches is agreeing, and must still get its repo/profile.
-    const pinned = projectOf(resolved.projectKey) ?? projectOf(resolved.issueKey);
-    const matched = projectOf(context.projectKey);
+    // A pinned project that disagrees with the match discards its repo/profile/issue together.
+    const pinned =
+      issueKeyToProjectKey(resolved.projectKey) ?? issueKeyToProjectKey(resolved.issueKey);
+    const matched = issueKeyToProjectKey(context.projectKey);
     const conflict = Boolean(pinned && matched && pinned !== matched);
 
     if (conflict) {

package/src/gatewayDataDir.js

@@ -20,7 +20,7 @@ export const APP_DIR_NAME = "devflow";
 /** Filename the daemon writes the gateway token to, under the data dir. */
 export const TOKEN_FILE = "gateway-token";
 
-/** Default gateway port — must match desktop/src/gateway/server.js. */
+/** Default gateway port, shared by the daemon and every client. */
 export const DEFAULT_PORT = 8787;
 
 /** Loopback host; the gateway only ever binds 127.0.0.1. */