claude-teammate 0.1.307 → 0.1.308

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-teammate",
-  "version": "0.1.307",
+  "version": "0.1.308",
   "description": "CLI bootstrapper for Claude Teammate.",
   "license": "MIT",
   "type": "module",

package/src/claude/prompts.js

@@ -179,6 +179,7 @@ export function buildJiraClarificationUserPrompt(input) {
     referenceRepos: input.referenceRepos,
     repoPaths: input.repoPaths
   });
+  const attachmentsSection = formatIssueAttachments(input.issue.attachments);
 
   return `Clarify this Jira issue.
 
@@ -189,6 +190,7 @@ Status: ${input.issue.status}
 Description:
 ${input.issue.descriptionText || "(none)"}
 
+${attachmentsSection}
 Memory snapshot:
 ${JSON.stringify(input.memory, null, 2)}
 
@@ -202,6 +204,20 @@ ${reopenContext}
 Decide whether the requirements are clear enough to plan implementation and whether the task needs code changes. Read code only if it helps.`;
 }
 
+// List attachments already present on the Jira issue so the agent reads
+// material the user attached instead of asking for it. These are downloadable
+// with jira_download_attachments by filename.
+function formatIssueAttachments(attachments) {
+  const list = (Array.isArray(attachments) ? attachments : []).filter((item) => item && item.filename);
+  if (list.length === 0) {
+    return "";
+  }
+  const lines = list.map((item) => `- ${item.filename}${item.mimeType ? ` (${item.mimeType})` : ""}`).join("\n");
+  return `Issue attachments (download with jira_download_attachments before planning; read every one relevant to the task):
+${lines}
+`;
+}
+
 function formatClarificationRepoScope({ targetRepo, referenceRepos, repoPaths }) {
   const repoPathLines = (Array.isArray(repoPaths) ? repoPaths : []).filter(Boolean);
 

package/src/jira.js

@@ -306,10 +306,28 @@ function mapIssueDetail(issue, baseUrl) {
     ? issue.fields.comment.comments.map(mapComment).sort(compareCommentsByCreated)
     : [];
 
+  // The detail fetch uses fields=*all, so existing attachments are already in
+  // the payload — surface them so the intake/planning step can download and
+  // read documents that users attach to the issue (a frequent "tài liệu đã
+  // đính kèm" complaint where the bot ignored material already on the task).
+  const attachments = Array.isArray(issue.fields?.attachment) ? issue.fields.attachment.map(mapAttachment) : [];
+
   return {
     ...mapIssue(issue, baseUrl),
     descriptionText: adfToText(issue.fields?.description).trim(),
-    comments
+    comments,
+    attachments
+  };
+}
+
+function mapAttachment(attachment) {
+  return {
+    id: attachment?.id ?? null,
+    filename: attachment?.filename ?? "",
+    mimeType: attachment?.mimeType ?? "",
+    size: attachment?.size ?? null,
+    url: attachment?.content ?? null,
+    created: attachment?.created ?? null
   };
 }
 

package/src/memory.js

@@ -263,6 +263,7 @@ function normalizeIssueMemoryData(data) {
   normalized.last_jira_memory_comment_id = String(normalized.last_jira_memory_comment_id ?? "").trim();
   normalized.code_change_verdict = String(normalized.code_change_verdict ?? "").trim();
   normalized.code_change_input_id = String(normalized.code_change_input_id ?? "").trim();
+  normalized.no_code_locked = Boolean(normalized.no_code_locked);
   delete normalized.github_issue_url;
   delete normalized.github_issue_number;
   delete normalized.latest_processed_comment_id;

package/src/worker/jira-helpers.js

@@ -185,11 +185,51 @@ export function shouldDisplayIssueInState(issue) {
   });
 }
 
+// Matches an explicit "no code" declaration in any of: "no code", "no_code",
+// "no-code", "nocode" (case-insensitive). Users add this to tell the bot a task
+// is documentation/estimate only and must NOT create a repo issue or push code.
+const NO_CODE_MARKER = /\bno[\s_-]?code\b/iu;
+
+// Deterministic no-code detection from explicit user signals, so the verdict no
+// longer depends on the LLM guessing (which flipped to "code" and made users
+// repeat "task no code" on every comment). Checks an explicit Jira label, the
+// description, and the latest human comment.
+export function detectNoCodeMarker(detail, botUser, jira) {
+  if (!detail) {
+    return false;
+  }
+
+  const labels = Array.isArray(detail.labels) ? detail.labels : [];
+  if (labels.some((label) => /^no[\s_-]?code$/iu.test(String(label || "").trim()))) {
+    return true;
+  }
+
+  if (NO_CODE_MARKER.test(String(detail.descriptionText || ""))) {
+    return true;
+  }
+
+  const latestHumanComment = [...(Array.isArray(detail.comments) ? detail.comments : [])]
+    .filter((comment) => !String(comment?.bodyText || "").startsWith(PROGRESS_COMMENT_PREFIX))
+    .sort(compareCommentsNewestFirst)
+    .find((comment) => (botUser ? !jira.isBotAuthor(comment.author, botUser) : true));
+
+  return Boolean(latestHumanComment && NO_CODE_MARKER.test(String(latestHumanComment.bodyText || "")));
+}
+
 export function shouldReuseNoCodeDecision({ issueMemory, latestInputId }) {
+  const noGithubIssues = !Array.isArray(issueMemory?.github_issues) || issueMemory.github_issues.length === 0;
+
+  // Once an explicit no-code marker has locked the issue, keep the no-code
+  // verdict sticky across subsequent comments (new input ids) so a follow-up
+  // that does not repeat "no code" is not re-evaluated back into a code task.
+  if (issueMemory?.no_code_locked && noGithubIssues) {
+    return true;
+  }
+
   return (
     issueMemory?.code_change_verdict === "no_code" &&
     issueMemory?.code_change_input_id === latestInputId &&
-    (!Array.isArray(issueMemory?.github_issues) || issueMemory.github_issues.length === 0)
+    noGithubIssues
   );
 }
 

package/src/worker/jira-issue-workflow.js

@@ -22,6 +22,7 @@ import {
   cleanupJiraProgressComments,
   deriveRawTaskRepos,
   deriveTaskRepos,
+  detectNoCodeMarker,
   ensureJiraComment,
   getLatestBlockedTaskRestartComment,
   getLatestClarificationInput,
@@ -632,7 +633,18 @@ export async function processJiraIssue({
       issueMemory,
       latestInputId: latestInput.id
     });
-    if (shouldReusePreservedNoCodeDecision) {
+    if (detectNoCodeMarker(detail, botUser, jira)) {
+      // Explicit user signal ("task no code" / no-code label) overrides the
+      // LLM verdict and locks the issue so later comments stay no-code. This
+      // stops the bot from re-deciding "code" and creating a repo issue, which
+      // forced users to repeat "task no code" on every comment.
+      decisionResult = { verdict: "no_code" };
+      issueMemory.no_code_locked = true;
+      await logger.info("Explicit no-code marker detected; forcing no_code verdict", {
+        issue: detail.key,
+        input: latestInput.id
+      });
+    } else if (shouldReusePreservedNoCodeDecision) {
       decisionResult = { verdict: "no_code" };
       await logger.info("Reusing preserved no-code decision", {
         issue: detail.key,