Merge pull request 'review-labels' (#14) from review-labels into master

Reviewed-on: #14

.github/issue_template/ai-refinement.md

@@ -16,4 +16,4 @@ How can it be fixed or improved?
 
 Steps to reproduce?
 
-Anything other useful information, logs or screenshots?
+Any other useful information? Logs, screenshots, steps to reproduce?

.github/pull_request_template.md

@@ -9,7 +9,5 @@
 
 ## Checklist
 <!-- Complete after creating PR -->
- - [ ] Linked issues
+ - [ ] Reviewed changes (or use `Review/AI` label)
- - [ ] Reviewed changes
  - [ ] Updated comments/documentation
- 

.github/workflows/code-review.yml

@@ -2,7 +2,7 @@ name: Code review
 
 on:
     pull_request:
-        types: [opened, synchronize, reopened]
+        types: [opened, synchronize, reopened, labeled]
 
 jobs:
     review:

.github/workflows/ticket-refinement.yml

@@ -3,6 +3,7 @@ name: Ticket refinement
 on:
     issues:
         types: [labeled]
+
 jobs:
     format:
         runs-on: ubuntu-latest

package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@ztimson/ai-agents",
-	"version": "0.1.1",
+	"version": "0.1.2",
 	"description": "AI agents",
 	"keywords": ["ai", "review"],
 	"author": "ztimson",

src/refine.mjs

@@ -42,7 +42,7 @@ dotenv.config({path: '.env.local', override: true, quiet: true, debug: false});
 	}
 
 	// Gather readme & template
-	let title = '', type = '', labels = [], readme = '', readmeP = path.join(process.cwd(), 'README.md');
+	let title = '', labels = [], readme = '', readmeP = path.join(process.cwd(), 'README.md');
 	if(fs.existsSync(readmeP)) readme = fs.readFileSync(readmeP, 'utf-8');
 	const template = p ? fs.readFileSync(p, 'utf-8') : `## Description
 
@@ -105,7 +105,7 @@ Implementation details, constraints, dependencies, design decisions
 			args: {label: {type: 'string', description: 'Label name', required: true}},
 			fn: async (args) => {
 				labels.push(args.label);
-				fetch(`${git}/api/v1/repos/${owner}/${repo}/issues/${ticket}/labels`, {
+				return await fetch(`${git}/api/v1/repos/${owner}/${repo}/issues/${ticket}/labels`, {
 					method: 'POST',
 					headers: {'Authorization': `token ${auth}`, 'Content-Type': 'application/json'},
 					body: `{"labels":["${args.label}"]}`
@@ -116,11 +116,11 @@ Implementation details, constraints, dependencies, design decisions
 
 **MANDATORY STEPS:**
 1. Call \`title\` tool EXACTLY ONCE in format: "[Module] - [Verb] [subject]" (example: Storage - fix file uploads)
-2. Identify one label from each group which best applies to the ticket: ${labelsReq.replace(',', ', ')}
+2. Identify one label from each group which best applies to the ticket: ${labelsReq.replaceAll(',', ', ')}
 3. Call the \`add_label\` tool ONCE FOR EVERY LABEL identified in the previous step
-4. Filter the following labels to any that apply to this ticket: ${labelsOpt.replace(',', ', ')}
+4. Filter the following labels to any that apply to this ticket: ${labelsOpt.replaceAll(',', ', ')}
 5. Call the \`add_label\` tool ONCE FOR EVERY LABEL identified in the previous step
-6. Output the new ticket description in formated markdown matching the following rules:
+6. Output the new ticket description in formatted markdown matching the following rules:
 
 **TEMPLATE RULES:**
 - Use ## headers (match template exactly)
@@ -181,8 +181,9 @@ Output ONLY markdown. No explanations, labels, or extra formatting.`});
 	const hasDuplicates = (await ai.language.ask(`ID: ${issueData.id}\nTitle: ${title}\n\`\`\`markdown\n${body}\n\`\`\``, {
 		system: `Your job is to identify duplicates. Respond ONLY with the duplicate's ID number or "NONE" if no match exists\n\n${dupes}`
 	}))?.pop()?.content;
+
 	// Handle duplicates
-	if(hasDuplicates && hasDuplicates !== 'NONE' && (dupeId = dupeIds.find(id => id === hasDuplicates.trim()))) {
+	if(hasDuplicates && !hasDuplicates.toUpperCase().includes('NONE') && (dupeId = dupeIds.find(id => id == hasDuplicates.trim())) != null && dupeId != issueData.id) {
 		await fetch(`${git}/api/v1/repos/${owner}/${repo}/issues/${ticket}/comments`, {
 			method: 'POST',
 			headers: {'Authorization': `token ${auth}`, 'Content-Type': 'application/json'},

src/review.mjs

@@ -19,12 +19,23 @@ dotenv.config({path: '.env.local', override: true, quiet: true, debug: false});
 		owner = process.env['GIT_OWNER'],
 		repo = process.env['GIT_REPO'],
 		auth = process.env['GIT_TOKEN'],
+		labelEnabled = process.env['LABEL_ENABLED'] || 'Review/AI',
 		pr = process.env['PULL_REQUEST'],
 		host = process.env['AI_HOST'],
 		model = process.env['AI_MODEL'],
 		token = process.env['AI_TOKEN'];
 
 	console.log(`Reviewing: ${root}\n`);
+	const info = await fetch(`${git}/api/v1/repos/${owner}/${repo}/pulls/${pr}`)
+		.then(async resp => {
+			if(resp.ok) return resp.json();
+			throw new Error(`${resp.status} ${await resp.text()}`);
+		});
+	if(!info.labels.some(l => l.name === labelEnabled)) {
+		console.log('Skipping');
+		return process.exit();
+	}
+
 	const branch = process.env['GIT_BRANCH'] || await $`cd ${root} && git symbolic-ref refs/remotes/origin/HEAD`;
 	const comments = [];
 	const commit = await $`cd ${root} && git log -1 --pretty=format:%H`;
@@ -53,7 +64,7 @@ dotenv.config({path: '.env.local', override: true, quiet: true, debug: false});
 		...options,
 		model: [host, model],
 		path: process.env['path'] || os.tmpdir(),
-		system: `You are a code reviewer. Analyze the git diff and use the \`recommend\` tool for EACH issue you find. You must call \`recommend\` exactly once for every bug or improvement opportunity directly related to changes. Ignore formatting recommendations. After making all recommendations, provide some concluding remarks about the overall state of the changes.${existingComments}`,
+		system: `You are a code reviewer. Analyze the git diff and use the \`recommend\` tool for EACH issue you find. You must call \`recommend\` exactly once for every bug or improvement opportunity directly related to changes. Ignore formatting recommendations. After making all recommendations, provide a quick 75 words or less sitrep.${existingComments}`,
 		tools: [{
 			name: 'read_file',
 			description: 'Read contents of a file',
@@ -87,7 +98,16 @@ dotenv.config({path: '.env.local', override: true, quiet: true, debug: false});
 		}]
 	});
 
-	const messages = await ai.language.ask(gitDiff);
+	const messages = await ai.language.ask(`Title: ${info.title || 'None'}
+Description:
+\`\`\`md
+${info.body || 'None'}
+\`\`\`
+
+Git Diff:
+\`\`\`
+${gitDiff}
+\`\`\``);
 	const summary = messages.pop().content;
 	if(git) {
 		const res = await fetch(`${git}/api/v1/repos/${owner}/${repo}/pulls/${pr}/reviews`, {