ai-dev-requirements 0.2.0 → 0.2.1

package/dist/index.cjs

@@ -188,7 +188,7 @@ function loadConfig(startDir) {
 }
 //#endregion
 //#region package.json
-var version = "0.2.0";
+var version = "0.2.1";
 //#endregion
 //#region ../../src/utils/map-status.ts
 const ONES_STATUS_MAP = {
@@ -693,6 +693,21 @@ function htmlToPlainText(html) {
 function getTaskDetailText(task) {
 	return task.descriptionText?.trim() || htmlToPlainText(task.desc_rich ?? task.description ?? "");
 }
+function extractHtmlImageReferences(html) {
+	return Array.from(html.matchAll(/<img\b[^>]*>/gi), (match) => {
+		const tag = match[0];
+		const srcMatch = tag.match(/\bsrc\s*=\s*(?:"([^"]*)"|'([^']*)')/i);
+		const resourceMatch = tag.match(/\bdata-uuid\s*=\s*(?:"([^"]*)"|'([^']*)')/i);
+		return {
+			tag,
+			src: (srcMatch?.[1] ?? srcMatch?.[2] ?? "").replace(/&amp;/gi, "&").trim(),
+			resourceUuid: (resourceMatch?.[1] ?? resourceMatch?.[2] ?? "").trim()
+		};
+	});
+}
+function containsInlineTaskImages(task) {
+	return [task.description, task.desc_rich].some((value) => typeof value === "string" && /<img\b/i.test(value)) || /\[(?:image|图片)\]/i.test(task.descriptionText ?? "");
+}
 function isRecord(value) {
 	return value !== null && typeof value === "object" && !Array.isArray(value);
 }
@@ -1131,7 +1146,11 @@ var OnesAdapter = class extends BaseAdapter {
 		return response.json();
 	}
 	async fetchRelatedActivities(taskKey) {
-		return (await this.onesql(RELATED_ACTIVITIES_QUERY, { key: taskKey }, "Task")).data?.task?.relatedActivities ?? [];
+		try {
+			return (await this.onesql(RELATED_ACTIVITIES_QUERY, { key: taskKey }, "Task")).data?.task?.relatedActivities ?? [];
+		} catch {
+			return [];
+		}
 	}
 	async searchTaskByNumber(taskNumber) {
 		const session = await this.login();
@@ -1316,30 +1335,92 @@ var OnesAdapter = class extends BaseAdapter {
 			return null;
 		}
 	}
+	getAttachmentResourceUuid(image) {
+		if (image.src) try {
+			const source = new URL(image.src, this.config.apiBase);
+			if (source.origin === new URL(this.config.apiBase).origin) {
+				const match = source.pathname.match(/\/res\/attachment\/([^/]+)$/);
+				const resourceUuid = match?.[1] ? decodeOnesPathIdentifier(match[1]) : null;
+				if (resourceUuid) return resourceUuid;
+			}
+		} catch {}
+		return image.resourceUuid;
+	}
 	/**
 	* Replace stale image URLs in HTML with fresh signed URLs from the attachment API.
-	* Extracts data-uuid from <img> tags and resolves fresh URLs in parallel.
+	* Prefer the resource identifier from the attachment URL because ONES data-uuid
+	* can identify the editor node instead of the underlying attachment.
 	*/
-	async refreshImageUrls(html) {
+	async refreshImageUrls(html, freshUrlCache = /* @__PURE__ */ new Map()) {
 		if (!html) return html;
-		const matches = Array.from(html.matchAll(/<img\s[^>]*data-uuid="([^"]+)"[^>]*>/g));
-		if (matches.length === 0) return html;
-		const replacements = await Promise.all(matches.map(async (match) => {
-			const dataUuid = match[1];
-			const freshUrl = await this.getAttachmentUrl(dataUuid);
+		const images = extractHtmlImageReferences(html).flatMap((image) => {
+			const resourceUuid = this.getAttachmentResourceUuid(image);
+			return resourceUuid ? [{
+				image,
+				resourceUuid
+			}] : [];
+		});
+		if (images.length === 0) return html;
+		const replacements = await Promise.all(images.map(async ({ image, resourceUuid }) => {
+			let freshUrl = freshUrlCache.get(resourceUuid);
+			if (!freshUrl) {
+				freshUrl = this.getAttachmentUrl(resourceUuid);
+				freshUrlCache.set(resourceUuid, freshUrl);
+			}
 			return {
-				fullMatch: match[0],
-				dataUuid,
-				freshUrl
+				fullMatch: image.tag,
+				freshUrl: await freshUrl
 			};
 		}));
 		let result = html;
-		for (const { fullMatch, freshUrl } of replacements) if (freshUrl) {
-			const updatedImg = fullMatch.replace(/src="[^"]*"/, `src="${freshUrl}"`);
+		for (const { fullMatch, freshUrl } of replacements) {
+			if (!freshUrl) continue;
+			const updatedImg = /\bsrc\s*=/i.test(fullMatch) ? fullMatch.replace(/\bsrc\s*=\s*(?:"[^"]*"|'[^']*')/i, `src="${freshUrl}"`) : fullMatch.replace(/<img\b/i, `<img src="${freshUrl}"`);
 			result = result.replace(fullMatch, updatedImg);
 		}
 		return result;
 	}
+	async getFreshTaskDescriptions(task) {
+		const taskInfo = await this.fetchTaskInfo(task.uuid);
+		const rawDescription = typeof taskInfo.desc === "string" ? taskInfo.desc : task.description ?? "";
+		const rawDescriptionRich = typeof taskInfo.desc_rich === "string" ? taskInfo.desc_rich : task.desc_rich ?? task.description ?? "";
+		const freshUrlCache = /* @__PURE__ */ new Map();
+		const [description, descriptionRich] = await Promise.all([this.refreshImageUrls(rawDescription, freshUrlCache), this.refreshImageUrls(rawDescriptionRich, freshUrlCache)]);
+		return {
+			description,
+			descriptionRich
+		};
+	}
+	async getTaskImageAttachments(task) {
+		const { description, descriptionRich } = await this.getFreshTaskDescriptions(task);
+		const images = [...extractHtmlImageReferences(descriptionRich), ...extractHtmlImageReferences(description)];
+		const seen = /* @__PURE__ */ new Set();
+		const attachments = [];
+		for (const image of images) {
+			if (!image.src) continue;
+			let url;
+			try {
+				url = new URL(image.src, this.config.apiBase).toString();
+			} catch {
+				continue;
+			}
+			if (this.classifyRemoteImageUrl(url) === "untrusted") continue;
+			const identity = image.resourceUuid || url;
+			if (seen.has(identity)) continue;
+			seen.add(identity);
+			const pathname = new URL(url).pathname;
+			const pathName = attachmentNameFromPath(pathname);
+			const name = pathName && pathName !== "/" ? pathName : `image-${attachments.length + 1}.png`;
+			attachments.push({
+				id: image.resourceUuid || `${task.uuid}-image-${attachments.length + 1}`,
+				name,
+				url,
+				mimeType: mimeTypeFromFileName(pathname),
+				size: 0
+			});
+		}
+		return attachments;
+	}
 	/**
 	* Fetch wiki page content via REST API.
 	* Endpoint: /wiki/api/wiki/team/{teamUuid}/online_page/{wikiUuid}/content
@@ -1459,7 +1540,7 @@ var OnesAdapter = class extends BaseAdapter {
 			title: `Wiki ${wikiUuid}`,
 			uuid: wikiUuid
 		});
-		const wikiContents = await Promise.all([...wikiRefs.values()].map(async (wiki) => {
+		const [wikiContents, taskImageAttachments] = await Promise.all([Promise.all([...wikiRefs.values()].map(async (wiki) => {
 			const rendered = await this.fetchWikiContent(wiki.uuid);
 			return {
 				title: wiki.title,
@@ -1467,7 +1548,7 @@ var OnesAdapter = class extends BaseAdapter {
 				content: rendered.content,
 				attachments: rendered.attachments
 			};
-		}));
+		})), containsInlineTaskImages(task) ? this.getTaskImageAttachments(task) : Promise.resolve([])]);
 		const parts = [];
 		parts.push(`# #${task.number} ${task.name}`);
 		parts.push("");
@@ -1527,7 +1608,7 @@ var OnesAdapter = class extends BaseAdapter {
 			parts.push(detailText);
 		}
 		const wikiAttachments = wikiContents.flatMap((wiki) => wiki.attachments);
-		const req = toRequirement(task, parts.join("\n"), wikiAttachments);
+		const req = toRequirement(task, parts.join("\n"), [...wikiAttachments, ...taskImageAttachments]);
 		req.raw = {
 			...req.raw,
 			relatedActivities,
@@ -1735,35 +1816,13 @@ var OnesAdapter = class extends BaseAdapter {
 		}));
 	}
 	async getIssueDetail(params) {
-		let issueKey;
-		const numMatch = params.issueId.match(/^#?(\d+)$/);
-		if (numMatch) {
-			const taskNumber = Number.parseInt(numMatch[1], 10);
-			const found = ((await this.graphql(SEARCH_TASKS_QUERY, {
-				groupBy: { tasks: {} },
-				groupOrderBy: null,
-				orderBy: { createTime: "DESC" },
-				filterGroup: [{ number_in: [taskNumber] }],
-				search: null,
-				pagination: {
-					limit: 10,
-					preciseCount: false
-				},
-				limit: 10
-			}, "group-task-data")).data?.buckets?.flatMap((b) => b.tasks ?? []) ?? []).find((t) => t.number === taskNumber);
-			if (!found) throw new Error(`ONES: Issue #${taskNumber} not found in current team`);
-			issueKey = `task-${found.uuid}`;
-		} else issueKey = params.issueId.startsWith("task-") ? params.issueId : `task-${params.issueId}`;
+		const { key: issueKey } = await this.resolveTaskRef(params.issueId);
 		const task = (await this.graphql(ISSUE_DETAIL_QUERY, { key: issueKey }, "Task")).data?.task;
 		if (!task) throw new Error(`ONES: Issue "${issueKey}" not found`);
 		const kind = classifyOnesWorkItem(task.issueType, task.subIssueType);
 		if (kind === "unknown") throw new Error(`ONES: Unable to classify "${params.issueId}" before get_issue_detail`);
 		if (kind === "requirement" || kind === "task") throw unsupportedWorkItemToolError(params.issueId, kind, "get_issue_detail", "get_work_item");
-		const taskInfo = await this.fetchTaskInfo(task.uuid);
-		const rawDescription = taskInfo.desc ?? task.description ?? "";
-		const rawDescRich = taskInfo.desc_rich ?? task.desc_rich ?? "";
-		const freshDescription = await this.refreshImageUrls(rawDescription);
-		const freshDescRich = await this.refreshImageUrls(rawDescRich);
+		const { description: freshDescription, descriptionRich: freshDescRich } = await this.getFreshTaskDescriptions(task);
 		return {
 			key: task.key,
 			uuid: task.uuid,
@@ -2356,7 +2415,7 @@ async function downloadTrustedImages(urls, options) {
 //#endregion
 //#region ../../src/tools/get-issue-detail.ts
 const GetIssueDetailSchema = zod_v4.z.object({
-	issueId: zod_v4.z.string().describe("The issue task ID or key (e.g. \"mock-issue-uuid\" or \"task-mock-issue-uuid\")"),
+	issueId: zod_v4.z.string().describe("ONES defect UUID, task key, number, or display ID (for example \"DEMO-2001\")"),
 	source: zod_v4.z.string().optional().describe("Source to fetch from. If omitted, uses the default source.")
 });
 /**

package/dist/index.mjs

@@ -165,7 +165,7 @@ function loadConfig(startDir) {
 }
 //#endregion
 //#region package.json
-var version = "0.2.0";
+var version = "0.2.1";
 //#endregion
 //#region ../../src/utils/map-status.ts
 const ONES_STATUS_MAP = {
@@ -670,6 +670,21 @@ function htmlToPlainText(html) {
 function getTaskDetailText(task) {
 	return task.descriptionText?.trim() || htmlToPlainText(task.desc_rich ?? task.description ?? "");
 }
+function extractHtmlImageReferences(html) {
+	return Array.from(html.matchAll(/<img\b[^>]*>/gi), (match) => {
+		const tag = match[0];
+		const srcMatch = tag.match(/\bsrc\s*=\s*(?:"([^"]*)"|'([^']*)')/i);
+		const resourceMatch = tag.match(/\bdata-uuid\s*=\s*(?:"([^"]*)"|'([^']*)')/i);
+		return {
+			tag,
+			src: (srcMatch?.[1] ?? srcMatch?.[2] ?? "").replace(/&amp;/gi, "&").trim(),
+			resourceUuid: (resourceMatch?.[1] ?? resourceMatch?.[2] ?? "").trim()
+		};
+	});
+}
+function containsInlineTaskImages(task) {
+	return [task.description, task.desc_rich].some((value) => typeof value === "string" && /<img\b/i.test(value)) || /\[(?:image|图片)\]/i.test(task.descriptionText ?? "");
+}
 function isRecord(value) {
 	return value !== null && typeof value === "object" && !Array.isArray(value);
 }
@@ -1108,7 +1123,11 @@ var OnesAdapter = class extends BaseAdapter {
 		return response.json();
 	}
 	async fetchRelatedActivities(taskKey) {
-		return (await this.onesql(RELATED_ACTIVITIES_QUERY, { key: taskKey }, "Task")).data?.task?.relatedActivities ?? [];
+		try {
+			return (await this.onesql(RELATED_ACTIVITIES_QUERY, { key: taskKey }, "Task")).data?.task?.relatedActivities ?? [];
+		} catch {
+			return [];
+		}
 	}
 	async searchTaskByNumber(taskNumber) {
 		const session = await this.login();
@@ -1293,30 +1312,92 @@ var OnesAdapter = class extends BaseAdapter {
 			return null;
 		}
 	}
+	getAttachmentResourceUuid(image) {
+		if (image.src) try {
+			const source = new URL(image.src, this.config.apiBase);
+			if (source.origin === new URL(this.config.apiBase).origin) {
+				const match = source.pathname.match(/\/res\/attachment\/([^/]+)$/);
+				const resourceUuid = match?.[1] ? decodeOnesPathIdentifier(match[1]) : null;
+				if (resourceUuid) return resourceUuid;
+			}
+		} catch {}
+		return image.resourceUuid;
+	}
 	/**
 	* Replace stale image URLs in HTML with fresh signed URLs from the attachment API.
-	* Extracts data-uuid from <img> tags and resolves fresh URLs in parallel.
+	* Prefer the resource identifier from the attachment URL because ONES data-uuid
+	* can identify the editor node instead of the underlying attachment.
 	*/
-	async refreshImageUrls(html) {
+	async refreshImageUrls(html, freshUrlCache = /* @__PURE__ */ new Map()) {
 		if (!html) return html;
-		const matches = Array.from(html.matchAll(/<img\s[^>]*data-uuid="([^"]+)"[^>]*>/g));
-		if (matches.length === 0) return html;
-		const replacements = await Promise.all(matches.map(async (match) => {
-			const dataUuid = match[1];
-			const freshUrl = await this.getAttachmentUrl(dataUuid);
+		const images = extractHtmlImageReferences(html).flatMap((image) => {
+			const resourceUuid = this.getAttachmentResourceUuid(image);
+			return resourceUuid ? [{
+				image,
+				resourceUuid
+			}] : [];
+		});
+		if (images.length === 0) return html;
+		const replacements = await Promise.all(images.map(async ({ image, resourceUuid }) => {
+			let freshUrl = freshUrlCache.get(resourceUuid);
+			if (!freshUrl) {
+				freshUrl = this.getAttachmentUrl(resourceUuid);
+				freshUrlCache.set(resourceUuid, freshUrl);
+			}
 			return {
-				fullMatch: match[0],
-				dataUuid,
-				freshUrl
+				fullMatch: image.tag,
+				freshUrl: await freshUrl
 			};
 		}));
 		let result = html;
-		for (const { fullMatch, freshUrl } of replacements) if (freshUrl) {
-			const updatedImg = fullMatch.replace(/src="[^"]*"/, `src="${freshUrl}"`);
+		for (const { fullMatch, freshUrl } of replacements) {
+			if (!freshUrl) continue;
+			const updatedImg = /\bsrc\s*=/i.test(fullMatch) ? fullMatch.replace(/\bsrc\s*=\s*(?:"[^"]*"|'[^']*')/i, `src="${freshUrl}"`) : fullMatch.replace(/<img\b/i, `<img src="${freshUrl}"`);
 			result = result.replace(fullMatch, updatedImg);
 		}
 		return result;
 	}
+	async getFreshTaskDescriptions(task) {
+		const taskInfo = await this.fetchTaskInfo(task.uuid);
+		const rawDescription = typeof taskInfo.desc === "string" ? taskInfo.desc : task.description ?? "";
+		const rawDescriptionRich = typeof taskInfo.desc_rich === "string" ? taskInfo.desc_rich : task.desc_rich ?? task.description ?? "";
+		const freshUrlCache = /* @__PURE__ */ new Map();
+		const [description, descriptionRich] = await Promise.all([this.refreshImageUrls(rawDescription, freshUrlCache), this.refreshImageUrls(rawDescriptionRich, freshUrlCache)]);
+		return {
+			description,
+			descriptionRich
+		};
+	}
+	async getTaskImageAttachments(task) {
+		const { description, descriptionRich } = await this.getFreshTaskDescriptions(task);
+		const images = [...extractHtmlImageReferences(descriptionRich), ...extractHtmlImageReferences(description)];
+		const seen = /* @__PURE__ */ new Set();
+		const attachments = [];
+		for (const image of images) {
+			if (!image.src) continue;
+			let url;
+			try {
+				url = new URL(image.src, this.config.apiBase).toString();
+			} catch {
+				continue;
+			}
+			if (this.classifyRemoteImageUrl(url) === "untrusted") continue;
+			const identity = image.resourceUuid || url;
+			if (seen.has(identity)) continue;
+			seen.add(identity);
+			const pathname = new URL(url).pathname;
+			const pathName = attachmentNameFromPath(pathname);
+			const name = pathName && pathName !== "/" ? pathName : `image-${attachments.length + 1}.png`;
+			attachments.push({
+				id: image.resourceUuid || `${task.uuid}-image-${attachments.length + 1}`,
+				name,
+				url,
+				mimeType: mimeTypeFromFileName(pathname),
+				size: 0
+			});
+		}
+		return attachments;
+	}
 	/**
 	* Fetch wiki page content via REST API.
 	* Endpoint: /wiki/api/wiki/team/{teamUuid}/online_page/{wikiUuid}/content
@@ -1436,7 +1517,7 @@ var OnesAdapter = class extends BaseAdapter {
 			title: `Wiki ${wikiUuid}`,
 			uuid: wikiUuid
 		});
-		const wikiContents = await Promise.all([...wikiRefs.values()].map(async (wiki) => {
+		const [wikiContents, taskImageAttachments] = await Promise.all([Promise.all([...wikiRefs.values()].map(async (wiki) => {
 			const rendered = await this.fetchWikiContent(wiki.uuid);
 			return {
 				title: wiki.title,
@@ -1444,7 +1525,7 @@ var OnesAdapter = class extends BaseAdapter {
 				content: rendered.content,
 				attachments: rendered.attachments
 			};
-		}));
+		})), containsInlineTaskImages(task) ? this.getTaskImageAttachments(task) : Promise.resolve([])]);
 		const parts = [];
 		parts.push(`# #${task.number} ${task.name}`);
 		parts.push("");
@@ -1504,7 +1585,7 @@ var OnesAdapter = class extends BaseAdapter {
 			parts.push(detailText);
 		}
 		const wikiAttachments = wikiContents.flatMap((wiki) => wiki.attachments);
-		const req = toRequirement(task, parts.join("\n"), wikiAttachments);
+		const req = toRequirement(task, parts.join("\n"), [...wikiAttachments, ...taskImageAttachments]);
 		req.raw = {
 			...req.raw,
 			relatedActivities,
@@ -1712,35 +1793,13 @@ var OnesAdapter = class extends BaseAdapter {
 		}));
 	}
 	async getIssueDetail(params) {
-		let issueKey;
-		const numMatch = params.issueId.match(/^#?(\d+)$/);
-		if (numMatch) {
-			const taskNumber = Number.parseInt(numMatch[1], 10);
-			const found = ((await this.graphql(SEARCH_TASKS_QUERY, {
-				groupBy: { tasks: {} },
-				groupOrderBy: null,
-				orderBy: { createTime: "DESC" },
-				filterGroup: [{ number_in: [taskNumber] }],
-				search: null,
-				pagination: {
-					limit: 10,
-					preciseCount: false
-				},
-				limit: 10
-			}, "group-task-data")).data?.buckets?.flatMap((b) => b.tasks ?? []) ?? []).find((t) => t.number === taskNumber);
-			if (!found) throw new Error(`ONES: Issue #${taskNumber} not found in current team`);
-			issueKey = `task-${found.uuid}`;
-		} else issueKey = params.issueId.startsWith("task-") ? params.issueId : `task-${params.issueId}`;
+		const { key: issueKey } = await this.resolveTaskRef(params.issueId);
 		const task = (await this.graphql(ISSUE_DETAIL_QUERY, { key: issueKey }, "Task")).data?.task;
 		if (!task) throw new Error(`ONES: Issue "${issueKey}" not found`);
 		const kind = classifyOnesWorkItem(task.issueType, task.subIssueType);
 		if (kind === "unknown") throw new Error(`ONES: Unable to classify "${params.issueId}" before get_issue_detail`);
 		if (kind === "requirement" || kind === "task") throw unsupportedWorkItemToolError(params.issueId, kind, "get_issue_detail", "get_work_item");
-		const taskInfo = await this.fetchTaskInfo(task.uuid);
-		const rawDescription = taskInfo.desc ?? task.description ?? "";
-		const rawDescRich = taskInfo.desc_rich ?? task.desc_rich ?? "";
-		const freshDescription = await this.refreshImageUrls(rawDescription);
-		const freshDescRich = await this.refreshImageUrls(rawDescRich);
+		const { description: freshDescription, descriptionRich: freshDescRich } = await this.getFreshTaskDescriptions(task);
 		return {
 			key: task.key,
 			uuid: task.uuid,
@@ -2333,7 +2392,7 @@ async function downloadTrustedImages(urls, options) {
 //#endregion
 //#region ../../src/tools/get-issue-detail.ts
 const GetIssueDetailSchema = z.object({
-	issueId: z.string().describe("The issue task ID or key (e.g. \"mock-issue-uuid\" or \"task-mock-issue-uuid\")"),
+	issueId: z.string().describe("ONES defect UUID, task key, number, or display ID (for example \"DEMO-2001\")"),
 	source: z.string().optional().describe("Source to fetch from. If omitted, uses the default source.")
 });
 /**