From 92d39b6b73d3192d20c5fdd3ebe3fc4d85ceefe0 Mon Sep 17 00:00:00 2001 From: Ammar Date: Wed, 8 Oct 2025 15:39:48 -0500 Subject: [PATCH] =?UTF-8?q?=F0=9F=A4=96=20ci:=20remove=20push=20trigger=20?= =?UTF-8?q?on=20main=20to=20avoid=20duplicate=20runs?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit - Remove 'push: branches: [main]' from CI and Build workflows - Keep only merge_group and pull_request triggers - Workflows now run once in merge queue, not twice (queue + post-merge) - Docs workflow keeps push trigger (has path filters, only deploys) - Also updated propose_plan tool to recommend best option when presenting choices --- .github/workflows/build.yml | 2 -- .github/workflows/ci.yml | 2 -- src/utils/tools/toolDefinitions.ts | 1 + 3 files changed, 1 insertion(+), 4 deletions(-) diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml index 551dbfec3..9e473ebb1 100644 --- a/.github/workflows/build.yml +++ b/.github/workflows/build.yml @@ -1,8 +1,6 @@ name: Build on: - push: - branches: ["main"] pull_request: branches: ["**"] merge_group: diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index c81ad7363..e043a26f2 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -1,8 +1,6 @@ name: CI on: - push: - branches: ["main"] pull_request: branches: ["**"] merge_group: diff --git a/src/utils/tools/toolDefinitions.ts b/src/utils/tools/toolDefinitions.ts index fced39a42..14cd4094e 100644 --- a/src/utils/tools/toolDefinitions.ts +++ b/src/utils/tools/toolDefinitions.ts @@ -143,6 +143,7 @@ export const TOOL_DEFINITIONS = { "Scale the detail to match the task complexity: for straightforward changes, briefly state what and why; " + "for complex changes, explain approach, key decisions, risks/tradeoffs; " + "for uncertain changes, clarify options and what needs user input. " + + "When presenting options, always provide your recommendation for the overall best option for the user. " + "For highly complex concepts, use mermaid diagrams where they'd clarify better than text. " + "Cover what's necessary to understand and approve the approach. Omit obvious details or ceremony." ),