tewip-ai 0.70.4 → 0.70.6

package/dist/engine/a2a/serve.js

@@ -112,15 +112,15 @@ export function handleRpc(cmd, body) {
     }
     const choice = chooseSkill(message);
     if (choice.kind === 'ask') {
-        record(cmd.root, { method, outcome: 'asked-back', why: choice.why });
+        record(cmd.root, { http: 'POST', path: JSONRPC_PATH, method, outcome: 'asked-back', why: choice.why });
         return { status: 200, payload: { jsonrpc: '2.0', id, result: needsInput(askWhich(choice.why)) } };
     }
     if (choice.kind === 'unknown') {
-        record(cmd.root, { method, outcome: 'unknown-skill', asked: choice.asked });
+        record(cmd.root, { http: 'POST', path: JSONRPC_PATH, method, outcome: 'unknown-skill', asked: choice.asked });
         return { status: 200, payload: { jsonrpc: '2.0', id, result: needsInput(askWhich(`"${choice.asked}" is not a skill Tewip has`)) } };
     }
     const answered = runSkill(cmd.root, choice.skill, []);
-    record(cmd.root, { method, outcome: 'answered', skill: choice.skill.id, verb: choice.skill.verb, failed: answered.failed });
+    record(cmd.root, { http: 'POST', path: JSONRPC_PATH, method, outcome: 'answered', skill: choice.skill.id, verb: choice.skill.verb, failed: answered.failed });
     return { status: 200, payload: { jsonrpc: '2.0', id, result: completedTask(answered.text, answered.failed) } };
 }
 /** True of an address that only this machine can reach. */
@@ -130,11 +130,26 @@ export function serveA2A(cmd) {
     const url = `http://${cmd.host}:${cmd.port}${JSONRPC_PATH}`;
     const server = createServer((req, res) => {
         const path = (req.url ?? '').split('?')[0];
+        // What reaches the port, not only what gets answered.
+        //
+        // `record` was called from the three places inside the answering path, so the log held
+        // successful answers and nothing else. Measured by sending four requests and finding one: a POST
+        // to the wrong path, a JSON-RPC call for a method this agent does not have, and a malformed body
+        // all left **no trace at all**. On a port with no authentication those are the entries worth
+        // keeping - a peer probing for methods is exactly what an operator needs to see - and the file's
+        // own comment promises "a record of what another agent asked for and what it was told". A
+        // rejected request is still something another agent asked for.
+        //
+        // The request's body is never recorded, only what it asked for and how it was answered, because
+        // a body from another agent can contain anything.
+        const seen = (outcome, extra = {}) => record(root, { http: req.method ?? '', path, outcome, ...extra });
         if (req.method === 'GET' && path === CARD_PATH) {
+            seen('card-read');
             send(res, 200, agentCard(url, cmd.tewipRoot));
             return;
         }
         if (req.method !== 'POST' || path !== JSONRPC_PATH) {
+            seen('no-such-endpoint');
             send(res, 404, { error: `nothing here. The card is at ${CARD_PATH} and calls go to ${JSONRPC_PATH}` });
             return;
         }
@@ -145,13 +160,23 @@ export function serveA2A(cmd) {
                 parsed = JSON.parse(raw);
             }
             catch {
+                seen('body-not-json');
                 send(res, 400, rpcError(null, INVALID_PARAMS, 'the body is not JSON'));
                 return;
             }
             const { status, payload } = handleRpc({ ...cmd, root }, parsed);
+            // A JSON-RPC error rides in a 200 response - the transport worked, the call did not - so the
+            // HTTP status cannot be the test. Checking it missed exactly the entry worth having: a peer
+            // asking for a method this agent does not implement.
+            const rpc = payload ?? {};
+            if (rpc.error)
+                seen('refused', { why: rpc.error.message ?? '' });
             send(res, status, payload);
         })
-            .catch((error) => send(res, 413, rpcError(null, INVALID_PARAMS, error.message)));
+            .catch((error) => {
+            seen('body-rejected', { why: error.message });
+            send(res, 413, rpcError(null, INVALID_PARAMS, error.message));
+        });
     });
     server.listen(cmd.port, cmd.host, () => {
         console.log(`Tewip is answering other agents about ${root}`);

package/dist/engine/work/jira-command.js

@@ -54,6 +54,35 @@ export async function runJira(cmd) {
         return;
     }
     const config = jiraFromEnv();
+    // Without a tracker, show what would be filed anyway.
+    //
+    // This file's own comment says "It shows what it would create and creates nothing until asked",
+    // and the help text says "Shows what it would create; --create actually files them" - but the
+    // refusal below ran whatever was asked, so a person who only wanted to see the list had to go and
+    // get a Jira account first. `bugs`, which does the same job for failures, has always answered this
+    // case: "No tracker was contacted: this is what would be filed."
+    //
+    // What a dry run genuinely cannot do without a server is say which scenarios are **already**
+    // filed, because that answer lives in the tracker. So it says so rather than implying the whole
+    // list is new work.
+    if (!config?.token && !cmd.create) {
+        const planned = JSON.parse(readFileSync(cmd.planFile, 'utf8'));
+        const wanted = chosen(planned.plan, cmd.only);
+        if (wanted.length === 0)
+            throw new Error(cmd.only.length ? `none of ${cmd.only.join(', ')} is in ${cmd.planFile}` : 'every scenario in the plan is already covered by a test');
+        console.log([
+            `Plan: ${planned.plan.title}`,
+            `Project: ${cmd.projectKey || '(none given)'}`,
+            '',
+            'No tracker was contacted: this is what would be filed.',
+            ...wanted.map((scenario) => `  ${scenario.id}  would create: ${scenario.id} ${scenario.title}`),
+            '',
+            `${wanted.length} item(s). Tewip could not check which of these you have already filed - that answer is in the tracker, not in the plan - so read the list as everything, not as new work.`,
+            'To file them, set a tracker up and add --create:',
+            jiraHelp(),
+        ].join(String.fromCharCode(10)));
+        return;
+    }
     if (!config)
         throw new Error(jiraHelp());
     if (!config.token)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tewip-ai",
-  "version": "0.70.4",
+  "version": "0.70.6",
   "mcpName": "io.github.hmamut39/tewip",
   "description": "Tewip finds why each failing test failed - real bug, flaky test, environment, test data or changed selector - for any stack, from the command line, CI, or your editor.",
   "keywords": [