diff --git a/.github/workflows/ask.lock.yml b/.github/workflows/ask.lock.yml index 9ecefea5a..bc5cc8d64 100644 --- a/.github/workflows/ask.lock.yml +++ b/.github/workflows/ask.lock.yml @@ -2,7 +2,7 @@ # To update this file, edit the corresponding .md file and run: # gh aw compile # -# Effective stop-time: 2025-09-19 14:50:09 +# Effective stop-time: 2025-09-19 15:41:02 name: "Question Answering Researcher" on: @@ -767,7 +767,7 @@ jobs: description: "Create a new GitHub pull request", inputSchema: { type: "object", - required: ["title", "body"], + required: ["title", "body", "branch"], properties: { title: { type: "string", description: "Pull request title" }, body: { @@ -776,8 +776,7 @@ jobs: }, branch: { type: "string", - description: - "Optional branch name (will be auto-generated if not provided)", + description: "Required branch name", }, labels: { type: "array", @@ -899,9 +898,9 @@ jobs: description: "Push changes to a pull request branch", inputSchema: { type: "object", - required: ["branch_name", "message"], + required: ["branch", "message"], properties: { - branch_name: { + branch: { type: "string", description: "The name of the branch to push to, should be the branch name associated with the pull request", @@ -1000,12 +999,19 @@ jobs: ? tool.inputSchema.required : []; if (requiredFields.length) { - const missing = requiredFields.filter(f => args[f] === undefined); + const missing = requiredFields.filter(f => { + const value = args[f]; + return ( + value === undefined || + value === null || + (typeof value === "string" && value.trim() === "") + ); + }); if (missing.length) { replyError( id, -32602, - `Invalid arguments: missing ${missing.map(m => `'${m}'`).join(", ")}` + `Invalid arguments: missing or empty ${missing.map(m => `'${m}'`).join(", ")}` ); return; } @@ -1072,7 +1078,7 @@ jobs: WORKFLOW_NAME="Question Answering Researcher" # Check stop-time limit - STOP_TIME="2025-09-19 14:50:09" + STOP_TIME="2025-09-19 15:41:02" echo "Checking stop-time limit: $STOP_TIME" # Convert stop time to epoch seconds @@ -1886,13 +1892,16 @@ jobs: ); continue; } + if (!item.branch || typeof item.branch !== "string") { + errors.push( + `Line ${i + 1}: create-pull-request requires a 'branch' string field` + ); + continue; + } // Sanitize text content item.title = sanitizeContent(item.title); item.body = sanitizeContent(item.body); - // Sanitize branch name if present - if (item.branch && typeof item.branch === "string") { - item.branch = sanitizeContent(item.branch); - } + item.branch = sanitizeContent(item.branch); // Sanitize labels if present if (item.labels && Array.isArray(item.labels)) { item.labels = item.labels.map( @@ -1989,10 +1998,10 @@ jobs: } break; case "push-to-pr-branch": - // Validate required branch_name field - if (!item.branch_name || typeof item.branch_name !== "string") { + // Validate required branch field + if (!item.branch || typeof item.branch !== "string") { errors.push( - `Line ${i + 1}: push-to-pr-branch requires a 'branch_name' string field` + `Line ${i + 1}: push-to-pr-branch requires a 'branch' string field` ); continue; } @@ -2004,7 +2013,7 @@ jobs: continue; } // Sanitize text content - item.branch_name = sanitizeContent(item.branch_name); + item.branch = sanitizeContent(item.branch); item.message = sanitizeContent(item.message); // Validate pull_request_number if provided (for target "*") const pushPRNumValidation = validateIssueOrPRNumber( diff --git a/.github/workflows/ci-doctor.lock.yml b/.github/workflows/ci-doctor.lock.yml index 325caeda3..ec13b464d 100644 --- a/.github/workflows/ci-doctor.lock.yml +++ b/.github/workflows/ci-doctor.lock.yml @@ -2,7 +2,7 @@ # To update this file, edit the corresponding .md file and run: # gh aw compile # -# Effective stop-time: 2025-09-19 14:50:09 +# Effective stop-time: 2025-09-19 15:41:02 name: "CI Failure Doctor" "on": @@ -248,7 +248,7 @@ jobs: description: "Create a new GitHub pull request", inputSchema: { type: "object", - required: ["title", "body"], + required: ["title", "body", "branch"], properties: { title: { type: "string", description: "Pull request title" }, body: { @@ -257,8 +257,7 @@ jobs: }, branch: { type: "string", - description: - "Optional branch name (will be auto-generated if not provided)", + description: "Required branch name", }, labels: { type: "array", @@ -380,9 +379,9 @@ jobs: description: "Push changes to a pull request branch", inputSchema: { type: "object", - required: ["branch_name", "message"], + required: ["branch", "message"], properties: { - branch_name: { + branch: { type: "string", description: "The name of the branch to push to, should be the branch name associated with the pull request", @@ -481,12 +480,19 @@ jobs: ? tool.inputSchema.required : []; if (requiredFields.length) { - const missing = requiredFields.filter(f => args[f] === undefined); + const missing = requiredFields.filter(f => { + const value = args[f]; + return ( + value === undefined || + value === null || + (typeof value === "string" && value.trim() === "") + ); + }); if (missing.length) { replyError( id, -32602, - `Invalid arguments: missing ${missing.map(m => `'${m}'`).join(", ")}` + `Invalid arguments: missing or empty ${missing.map(m => `'${m}'`).join(", ")}` ); return; } @@ -553,7 +559,7 @@ jobs: WORKFLOW_NAME="CI Failure Doctor" # Check stop-time limit - STOP_TIME="2025-09-19 14:50:09" + STOP_TIME="2025-09-19 15:41:02" echo "Checking stop-time limit: $STOP_TIME" # Convert stop time to epoch seconds @@ -1500,13 +1506,16 @@ jobs: ); continue; } + if (!item.branch || typeof item.branch !== "string") { + errors.push( + `Line ${i + 1}: create-pull-request requires a 'branch' string field` + ); + continue; + } // Sanitize text content item.title = sanitizeContent(item.title); item.body = sanitizeContent(item.body); - // Sanitize branch name if present - if (item.branch && typeof item.branch === "string") { - item.branch = sanitizeContent(item.branch); - } + item.branch = sanitizeContent(item.branch); // Sanitize labels if present if (item.labels && Array.isArray(item.labels)) { item.labels = item.labels.map( @@ -1603,10 +1612,10 @@ jobs: } break; case "push-to-pr-branch": - // Validate required branch_name field - if (!item.branch_name || typeof item.branch_name !== "string") { + // Validate required branch field + if (!item.branch || typeof item.branch !== "string") { errors.push( - `Line ${i + 1}: push-to-pr-branch requires a 'branch_name' string field` + `Line ${i + 1}: push-to-pr-branch requires a 'branch' string field` ); continue; } @@ -1618,7 +1627,7 @@ jobs: continue; } // Sanitize text content - item.branch_name = sanitizeContent(item.branch_name); + item.branch = sanitizeContent(item.branch); item.message = sanitizeContent(item.message); // Validate pull_request_number if provided (for target "*") const pushPRNumValidation = validateIssueOrPRNumber( diff --git a/.github/workflows/daily-backlog-burner.lock.yml b/.github/workflows/daily-backlog-burner.lock.yml index 6ef826d57..8a591d1ce 100644 --- a/.github/workflows/daily-backlog-burner.lock.yml +++ b/.github/workflows/daily-backlog-burner.lock.yml @@ -2,7 +2,7 @@ # To update this file, edit the corresponding .md file and run: # gh aw compile # -# Effective stop-time: 2025-09-19 14:50:09 +# Effective stop-time: 2025-09-19 15:41:02 name: "Daily Backlog Burner" "on": @@ -228,7 +228,7 @@ jobs: description: "Create a new GitHub pull request", inputSchema: { type: "object", - required: ["title", "body"], + required: ["title", "body", "branch"], properties: { title: { type: "string", description: "Pull request title" }, body: { @@ -237,8 +237,7 @@ jobs: }, branch: { type: "string", - description: - "Optional branch name (will be auto-generated if not provided)", + description: "Required branch name", }, labels: { type: "array", @@ -360,9 +359,9 @@ jobs: description: "Push changes to a pull request branch", inputSchema: { type: "object", - required: ["branch_name", "message"], + required: ["branch", "message"], properties: { - branch_name: { + branch: { type: "string", description: "The name of the branch to push to, should be the branch name associated with the pull request", @@ -461,12 +460,19 @@ jobs: ? tool.inputSchema.required : []; if (requiredFields.length) { - const missing = requiredFields.filter(f => args[f] === undefined); + const missing = requiredFields.filter(f => { + const value = args[f]; + return ( + value === undefined || + value === null || + (typeof value === "string" && value.trim() === "") + ); + }); if (missing.length) { replyError( id, -32602, - `Invalid arguments: missing ${missing.map(m => `'${m}'`).join(", ")}` + `Invalid arguments: missing or empty ${missing.map(m => `'${m}'`).join(", ")}` ); return; } @@ -533,7 +539,7 @@ jobs: WORKFLOW_NAME="Daily Backlog Burner" # Check stop-time limit - STOP_TIME="2025-09-19 14:50:09" + STOP_TIME="2025-09-19 15:41:02" echo "Checking stop-time limit: $STOP_TIME" # Convert stop time to epoch seconds @@ -1413,13 +1419,16 @@ jobs: ); continue; } + if (!item.branch || typeof item.branch !== "string") { + errors.push( + `Line ${i + 1}: create-pull-request requires a 'branch' string field` + ); + continue; + } // Sanitize text content item.title = sanitizeContent(item.title); item.body = sanitizeContent(item.body); - // Sanitize branch name if present - if (item.branch && typeof item.branch === "string") { - item.branch = sanitizeContent(item.branch); - } + item.branch = sanitizeContent(item.branch); // Sanitize labels if present if (item.labels && Array.isArray(item.labels)) { item.labels = item.labels.map( @@ -1516,10 +1525,10 @@ jobs: } break; case "push-to-pr-branch": - // Validate required branch_name field - if (!item.branch_name || typeof item.branch_name !== "string") { + // Validate required branch field + if (!item.branch || typeof item.branch !== "string") { errors.push( - `Line ${i + 1}: push-to-pr-branch requires a 'branch_name' string field` + `Line ${i + 1}: push-to-pr-branch requires a 'branch' string field` ); continue; } @@ -1531,7 +1540,7 @@ jobs: continue; } // Sanitize text content - item.branch_name = sanitizeContent(item.branch_name); + item.branch = sanitizeContent(item.branch); item.message = sanitizeContent(item.message); // Validate pull_request_number if provided (for target "*") const pushPRNumValidation = validateIssueOrPRNumber( diff --git a/.github/workflows/daily-perf-improver.lock.yml b/.github/workflows/daily-perf-improver.lock.yml index 66d44167c..3413207a4 100644 --- a/.github/workflows/daily-perf-improver.lock.yml +++ b/.github/workflows/daily-perf-improver.lock.yml @@ -2,7 +2,7 @@ # To update this file, edit the corresponding .md file and run: # gh aw compile # -# Effective stop-time: 2025-09-19 14:50:09 +# Effective stop-time: 2025-09-19 15:41:02 name: "Daily Perf Improver" "on": @@ -242,7 +242,7 @@ jobs: description: "Create a new GitHub pull request", inputSchema: { type: "object", - required: ["title", "body"], + required: ["title", "body", "branch"], properties: { title: { type: "string", description: "Pull request title" }, body: { @@ -251,8 +251,7 @@ jobs: }, branch: { type: "string", - description: - "Optional branch name (will be auto-generated if not provided)", + description: "Required branch name", }, labels: { type: "array", @@ -374,9 +373,9 @@ jobs: description: "Push changes to a pull request branch", inputSchema: { type: "object", - required: ["branch_name", "message"], + required: ["branch", "message"], properties: { - branch_name: { + branch: { type: "string", description: "The name of the branch to push to, should be the branch name associated with the pull request", @@ -475,12 +474,19 @@ jobs: ? tool.inputSchema.required : []; if (requiredFields.length) { - const missing = requiredFields.filter(f => args[f] === undefined); + const missing = requiredFields.filter(f => { + const value = args[f]; + return ( + value === undefined || + value === null || + (typeof value === "string" && value.trim() === "") + ); + }); if (missing.length) { replyError( id, -32602, - `Invalid arguments: missing ${missing.map(m => `'${m}'`).join(", ")}` + `Invalid arguments: missing or empty ${missing.map(m => `'${m}'`).join(", ")}` ); return; } @@ -547,7 +553,7 @@ jobs: WORKFLOW_NAME="Daily Perf Improver" # Check stop-time limit - STOP_TIME="2025-09-19 14:50:09" + STOP_TIME="2025-09-19 15:41:02" echo "Checking stop-time limit: $STOP_TIME" # Convert stop time to epoch seconds @@ -1488,13 +1494,16 @@ jobs: ); continue; } + if (!item.branch || typeof item.branch !== "string") { + errors.push( + `Line ${i + 1}: create-pull-request requires a 'branch' string field` + ); + continue; + } // Sanitize text content item.title = sanitizeContent(item.title); item.body = sanitizeContent(item.body); - // Sanitize branch name if present - if (item.branch && typeof item.branch === "string") { - item.branch = sanitizeContent(item.branch); - } + item.branch = sanitizeContent(item.branch); // Sanitize labels if present if (item.labels && Array.isArray(item.labels)) { item.labels = item.labels.map( @@ -1591,10 +1600,10 @@ jobs: } break; case "push-to-pr-branch": - // Validate required branch_name field - if (!item.branch_name || typeof item.branch_name !== "string") { + // Validate required branch field + if (!item.branch || typeof item.branch !== "string") { errors.push( - `Line ${i + 1}: push-to-pr-branch requires a 'branch_name' string field` + `Line ${i + 1}: push-to-pr-branch requires a 'branch' string field` ); continue; } @@ -1606,7 +1615,7 @@ jobs: continue; } // Sanitize text content - item.branch_name = sanitizeContent(item.branch_name); + item.branch = sanitizeContent(item.branch); item.message = sanitizeContent(item.message); // Validate pull_request_number if provided (for target "*") const pushPRNumValidation = validateIssueOrPRNumber( diff --git a/.github/workflows/daily-test-improver.lock.yml b/.github/workflows/daily-test-improver.lock.yml index 9e136876f..dea3b161f 100644 --- a/.github/workflows/daily-test-improver.lock.yml +++ b/.github/workflows/daily-test-improver.lock.yml @@ -2,7 +2,7 @@ # To update this file, edit the corresponding .md file and run: # gh aw compile # -# Effective stop-time: 2025-09-19 14:50:09 +# Effective stop-time: 2025-09-19 15:41:02 name: "Daily Test Coverage Improver" "on": @@ -242,7 +242,7 @@ jobs: description: "Create a new GitHub pull request", inputSchema: { type: "object", - required: ["title", "body"], + required: ["title", "body", "branch"], properties: { title: { type: "string", description: "Pull request title" }, body: { @@ -251,8 +251,7 @@ jobs: }, branch: { type: "string", - description: - "Optional branch name (will be auto-generated if not provided)", + description: "Required branch name", }, labels: { type: "array", @@ -374,9 +373,9 @@ jobs: description: "Push changes to a pull request branch", inputSchema: { type: "object", - required: ["branch_name", "message"], + required: ["branch", "message"], properties: { - branch_name: { + branch: { type: "string", description: "The name of the branch to push to, should be the branch name associated with the pull request", @@ -475,12 +474,19 @@ jobs: ? tool.inputSchema.required : []; if (requiredFields.length) { - const missing = requiredFields.filter(f => args[f] === undefined); + const missing = requiredFields.filter(f => { + const value = args[f]; + return ( + value === undefined || + value === null || + (typeof value === "string" && value.trim() === "") + ); + }); if (missing.length) { replyError( id, -32602, - `Invalid arguments: missing ${missing.map(m => `'${m}'`).join(", ")}` + `Invalid arguments: missing or empty ${missing.map(m => `'${m}'`).join(", ")}` ); return; } @@ -547,7 +553,7 @@ jobs: WORKFLOW_NAME="Daily Test Coverage Improver" # Check stop-time limit - STOP_TIME="2025-09-19 14:50:09" + STOP_TIME="2025-09-19 15:41:02" echo "Checking stop-time limit: $STOP_TIME" # Convert stop time to epoch seconds @@ -1463,13 +1469,16 @@ jobs: ); continue; } + if (!item.branch || typeof item.branch !== "string") { + errors.push( + `Line ${i + 1}: create-pull-request requires a 'branch' string field` + ); + continue; + } // Sanitize text content item.title = sanitizeContent(item.title); item.body = sanitizeContent(item.body); - // Sanitize branch name if present - if (item.branch && typeof item.branch === "string") { - item.branch = sanitizeContent(item.branch); - } + item.branch = sanitizeContent(item.branch); // Sanitize labels if present if (item.labels && Array.isArray(item.labels)) { item.labels = item.labels.map( @@ -1566,10 +1575,10 @@ jobs: } break; case "push-to-pr-branch": - // Validate required branch_name field - if (!item.branch_name || typeof item.branch_name !== "string") { + // Validate required branch field + if (!item.branch || typeof item.branch !== "string") { errors.push( - `Line ${i + 1}: push-to-pr-branch requires a 'branch_name' string field` + `Line ${i + 1}: push-to-pr-branch requires a 'branch' string field` ); continue; } @@ -1581,7 +1590,7 @@ jobs: continue; } // Sanitize text content - item.branch_name = sanitizeContent(item.branch_name); + item.branch = sanitizeContent(item.branch); item.message = sanitizeContent(item.message); // Validate pull_request_number if provided (for target "*") const pushPRNumValidation = validateIssueOrPRNumber( diff --git a/.github/workflows/pr-fix.lock.yml b/.github/workflows/pr-fix.lock.yml index 0aeef0189..b46e40abb 100644 --- a/.github/workflows/pr-fix.lock.yml +++ b/.github/workflows/pr-fix.lock.yml @@ -2,7 +2,7 @@ # To update this file, edit the corresponding .md file and run: # gh aw compile # -# Effective stop-time: 2025-09-19 14:50:09 +# Effective stop-time: 2025-09-19 15:41:02 name: "PR Fix" on: @@ -772,7 +772,7 @@ jobs: description: "Create a new GitHub pull request", inputSchema: { type: "object", - required: ["title", "body"], + required: ["title", "body", "branch"], properties: { title: { type: "string", description: "Pull request title" }, body: { @@ -781,8 +781,7 @@ jobs: }, branch: { type: "string", - description: - "Optional branch name (will be auto-generated if not provided)", + description: "Required branch name", }, labels: { type: "array", @@ -904,9 +903,9 @@ jobs: description: "Push changes to a pull request branch", inputSchema: { type: "object", - required: ["branch_name", "message"], + required: ["branch", "message"], properties: { - branch_name: { + branch: { type: "string", description: "The name of the branch to push to, should be the branch name associated with the pull request", @@ -1005,12 +1004,19 @@ jobs: ? tool.inputSchema.required : []; if (requiredFields.length) { - const missing = requiredFields.filter(f => args[f] === undefined); + const missing = requiredFields.filter(f => { + const value = args[f]; + return ( + value === undefined || + value === null || + (typeof value === "string" && value.trim() === "") + ); + }); if (missing.length) { replyError( id, -32602, - `Invalid arguments: missing ${missing.map(m => `'${m}'`).join(", ")}` + `Invalid arguments: missing or empty ${missing.map(m => `'${m}'`).join(", ")}` ); return; } @@ -1077,7 +1083,7 @@ jobs: WORKFLOW_NAME="PR Fix" # Check stop-time limit - STOP_TIME="2025-09-19 14:50:09" + STOP_TIME="2025-09-19 15:41:02" echo "Checking stop-time limit: $STOP_TIME" # Convert stop time to epoch seconds @@ -1917,13 +1923,16 @@ jobs: ); continue; } + if (!item.branch || typeof item.branch !== "string") { + errors.push( + `Line ${i + 1}: create-pull-request requires a 'branch' string field` + ); + continue; + } // Sanitize text content item.title = sanitizeContent(item.title); item.body = sanitizeContent(item.body); - // Sanitize branch name if present - if (item.branch && typeof item.branch === "string") { - item.branch = sanitizeContent(item.branch); - } + item.branch = sanitizeContent(item.branch); // Sanitize labels if present if (item.labels && Array.isArray(item.labels)) { item.labels = item.labels.map( @@ -2020,10 +2029,10 @@ jobs: } break; case "push-to-pr-branch": - // Validate required branch_name field - if (!item.branch_name || typeof item.branch_name !== "string") { + // Validate required branch field + if (!item.branch || typeof item.branch !== "string") { errors.push( - `Line ${i + 1}: push-to-pr-branch requires a 'branch_name' string field` + `Line ${i + 1}: push-to-pr-branch requires a 'branch' string field` ); continue; } @@ -2035,7 +2044,7 @@ jobs: continue; } // Sanitize text content - item.branch_name = sanitizeContent(item.branch_name); + item.branch = sanitizeContent(item.branch); item.message = sanitizeContent(item.message); // Validate pull_request_number if provided (for target "*") const pushPRNumValidation = validateIssueOrPRNumber(