Skip to content

Commit b984241

Browse files
alpslaclaude
andcommitted
fix(orchestrators): Update stale Semgrep comments in TypeScript orchestrator
SESSION 113: Consistent comment style across all orchestrators. Verified all 8 language orchestrators have correct Semgrep handling: - Go, PHP, Ruby, Rust, .NET: Never had the bug (no tier filtering) - TypeScript: Already had fix, updated stale method comment - Java, Python: Fixed in previous commit Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 9ee69df commit b984241

1 file changed

Lines changed: 2 additions & 3 deletions

File tree

packages/agents/src/two-branch/tools/typescript/typescript-tool-orchestrator.ts

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -302,9 +302,8 @@ export class TypeScriptToolOrchestrator extends BaseToolOrchestrator {
302302
/**
303303
* Get tools to run based on analysis mode (required by base)
304304
*
305-
* SESSION 34 OPTIMIZATION: userTier parameter for Semgrep skip logic
306-
* - BASIC tier: Run Semgrep here (Step 3), Lite Security Agent groups issues
307-
* - PRO tier: Skip Semgrep here, run scan+fix combined in Step 5.5
305+
* SESSION 34 FIX: Semgrep runs for ALL tiers in Step 3
306+
* (Original PRO tier optimization was incomplete - scan-fix-executor never ran Semgrep)
308307
*
309308
* CodeQL: Only runs if:
310309
* 1. config.codeql.enabled = true (explicit opt-in)

0 commit comments

Comments
 (0)