diff --git a/agents/instructions.ts b/agents/instructions.ts index 1d3651e..3e5dcd4 100644 --- a/agents/instructions.ts +++ b/agents/instructions.ts @@ -170,6 +170,8 @@ Tool names may be formatted as \`(server name)/(tool name)\`, for example: \`${g **Efficiency**: Trust the tools - do not repeatedly verify file contents or git status after operations. If a tool reports success, proceed to the next step. Only verify if you encounter an actual error. +**Command execution**: Never use \`sleep\` to wait for commands to complete. Commands run synchronously - when the bash tool returns, the command has finished. + **Commenting style**: When posting comments via ${ghPullfrogMcpName}, write as a professional team member would. Your final comments should be polished and actionableโ€”do not include intermediate reasoning like "I'll now look at the code" or "Let me respond to the question." **If you get stuck**: If you cannot complete a task due to missing information, ambiguity, or an unrecoverable error: diff --git a/entry b/entry index 1735fa2..c688ed8 100755 --- a/entry +++ b/entry @@ -83328,7 +83328,7 @@ function query({ // package.json var package_default = { name: "@pullfrog/action", - version: "0.0.147", + version: "0.0.148", type: "module", files: [ "index.js", @@ -92833,6 +92833,8 @@ Tool names may be formatted as \`(server name)/(tool name)\`, for example: \`${g **Efficiency**: Trust the tools - do not repeatedly verify file contents or git status after operations. If a tool reports success, proceed to the next step. Only verify if you encounter an actual error. +**Command execution**: Never use \`sleep\` to wait for commands to complete. Commands run synchronously - when the bash tool returns, the command has finished. + **Commenting style**: When posting comments via ${ghPullfrogMcpName}, write as a professional team member would. Your final comments should be polished and actionable\u2014do not include intermediate reasoning like "I'll now look at the code" or "Let me respond to the question." **If you get stuck**: If you cannot complete a task due to missing information, ambiguity, or an unrecoverable error: @@ -125111,11 +125113,11 @@ async function main(inputs) { apiKeys: apiKeySetup.apiKeys }; if (ctx.payload.event.trigger === "fix_review" && Array.isArray(ctx.payload.event.comment_ids) && ctx.payload.event.comment_ids.length === 0) { - await reportProgress(ctx, { - body: `\u{1F44D} **No approved comments found** + const noThumbsMessage = `\u{1F44D} **No approved comments found** -To use "Fix \u{1F44D}s", add a \u{1F44D} reaction to one or more inline review comments you want fixed.` - }); +To use "Fix \u{1F44D}s", add a \u{1F44D} reaction to one or more inline review comments you want fixed.`; + log.error(noThumbsMessage); + await reportProgress(ctx, { body: noThumbsMessage }); return { success: true }; } const result = await runAgent(ctx); diff --git a/main.ts b/main.ts index eaa606b..64f350c 100644 --- a/main.ts +++ b/main.ts @@ -195,9 +195,9 @@ export async function main(inputs: Inputs): Promise { Array.isArray(ctx.payload.event.comment_ids) && ctx.payload.event.comment_ids.length === 0 ) { - await reportProgress(ctx, { - body: `๐Ÿ‘ **No approved comments found**\n\nTo use "Fix ๐Ÿ‘s", add a ๐Ÿ‘ reaction to one or more inline review comments you want fixed.`, - }); + const noThumbsMessage = `๐Ÿ‘ **No approved comments found**\n\nTo use "Fix ๐Ÿ‘s", add a ๐Ÿ‘ reaction to one or more inline review comments you want fixed.`; + log.error(noThumbsMessage); + await reportProgress(ctx, { body: noThumbsMessage }); return { success: true }; } diff --git a/package.json b/package.json index 20f5e2c..85ee251 100644 --- a/package.json +++ b/package.json @@ -1,6 +1,6 @@ { "name": "@pullfrog/action", - "version": "0.0.147", + "version": "0.0.148", "type": "module", "files": [ "index.js",