Skip to content

chore: anchor Claude verdict marker filter to line-start #8835

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
aixaCode wants to merge 6 commits into
base: dev
Choose a base branch
from
Open

chore: anchor Claude verdict marker filter to line-start #8835

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
aac82dd
chore: hide stale Claude review comments instead of deleting
aixaCode May 20, 2026
e19554f
chore: scope claude-review concurrency per event type
aixaCode May 20, 2026
68e72d3
chore: move claude-review concurrency to job level
aixaCode May 20, 2026
9580511
chore: polish master-of-bots docs wording
aixaCode May 20, 2026
94e0f60
Merge branch 'dev' into chore/test-claude-review-workflow
aixaCode May 20, 2026
7bab08c
chore: anchor verdict marker filter to line-start
aixaCode May 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions .github/workflows/claude-pr-review.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -133,8 +133,9 @@ jobs:
owner, repo, issue_number: prNumber, per_page: 100
});

const VERDICT_RE = /(?:^|\n)REVIEW_RESULT: (PASS|FAIL)\b/;
const claudeComments = comments.data.filter(c =>
c.user?.login === 'claude[bot]' && c.body?.includes('REVIEW_RESULT:')
c.user?.login === 'claude[bot]' && VERDICT_RE.test(c.body || '')
);
const claudeComment = claudeComments[claudeComments.length - 1];

Expand Down Expand Up @@ -431,8 +432,9 @@ jobs:
per_page: 100
});

const VERDICT_RE = /(?:^|\n)REVIEW_RESULT: (PASS|FAIL)\b/;
const claudeComments = comments.data.filter(c =>
c.user?.login === 'claude[bot]' && c.body?.includes('REVIEW_RESULT:')
c.user?.login === 'claude[bot]' && VERDICT_RE.test(c.body || '')
);
const claudeComment = claudeComments[claudeComments.length - 1];

Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/dependency-security-review.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -186,8 +186,9 @@ jobs:
issue_number: ${{ github.event.pull_request.number }},
});

const VERDICT_RE = /(?:^|\n)DEPENDENCY_REVIEW: (PASS|BLOCK|NEEDS_ATTENTION)\b/;
const claudeComments = comments.filter(c =>
c.user?.login === 'claude[bot]' && c.body?.includes('DEPENDENCY_REVIEW:')
c.user?.login === 'claude[bot]' && VERDICT_RE.test(c.body || '')
);
const claudeComment = claudeComments[claudeComments.length - 1];

Expand Down
2 changes: 1 addition & 1 deletion docs/master-of-bots.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
# Master of Bots

Master Of Bots allows you to simulate big amounts (tested with up to 100 bots from a single machine) of users with real wallets for debug purposes.
Master Of Bots allows you to simulate large numbers (tested with up to 100 bots from a single machine) of users with real wallets for debug purposes.

Proceed to https://github.com/decentraland/livekit-bots/tree/explorer-alpha and follow the instructions.
Loading