- 13 2月, 2026 21 次提交
-
-
由 Will Chen 提交于
## Summary - add local `.agents/skills` symlink for skill tool access in workspace ## Test plan - npm run fmt - npm run lint:fix - npm run ts - npm test
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2690" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> -
由 Will Chen 提交于
## Summary - Adds comprehensive planning document for cloud sandbox runtime mode feature - Generated from collaborative planning session using dyad:swarm-to-plan skill - Outlines complete feature design, UX flows, technical architecture, and implementation phases ## Test plan - Document is for planning purposes and doesn't require functional testing - Review content for completeness and accuracy of scope and technical approach - Validate that implementation phases are well-sequenced and address all scope items
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2675" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> -
由 wwwillchen-bot 提交于
Fixes #1225 ## Summary - Separates the release workflow into build and publish phases - Build phase runs on all platforms in dry-run mode (no actual publish) - Artifacts are uploaded and downloaded between phases - Publish phase uses GitHub environment "release" for manual approval gate - Removes commented-out code for cleaner workflow ## Test plan - [ ] Trigger the release workflow manually - [ ] Verify build phase completes for all platforms (Windows, macOS, Linux) - [ ] Verify artifacts are uploaded successfully - [ ] Verify publish phase waits for environment approval - [ ] Approve and verify publish completes from dry-run artifacts - [ ] Verify verify-assets step still validates the release
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2662" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Adds a manual approval gate to the release workflow by splitting it into build (dry-run) and publish phases. Builds run on Windows, macOS, and Linux; publish reuses artifacts after approval via the “release” environment. - **New Features** - Build job runs npm run publish -- --dry-run on all OS and uploads out/ artifacts (1-day retention). - Publish job waits on GitHub environment “release”, downloads artifacts, and runs npm run publish -- --from-dry-run. - verify-assets now runs after publish to validate the release. - Cleaned up commented code and pins npm 11.8.0 in publish job. - **Migration** - Ensure the GitHub environment “release” exists and requires the desired approvers. <sup>Written for commit 04fcc4025e533a6d8fb0804df80c10b3215cff8a. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> Co-authored-by:Will Chen <willchen90@gmail.com> Co-authored-by:
Claude Opus 4.5 <noreply@anthropic.com>
-
由 Will Chen 提交于
#skip-bb <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2678" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end -->
-
由 Will Chen 提交于
## Summary - Updates pr-fix-comments skill to leverage product principles for resolving ambiguous feedback - Introduces 'Resolved by product principles' category to reduce human review burden - Adds guidance for citing principles in replies and flagging when insufficient - Enhances pr-fix meta-skill documentation on product principles usage ## Test plan 1. Verify that the updated skill documentation is accessible and clear 2. Review the new guidance on consulting rules/product-principles.md 3. Confirm the categorization options are correctly documented 4. Check that the reply examples include proper principle citations
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2674" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> -
由 Will Chen 提交于
<!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2672" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end -->
-
由 Will Chen 提交于
## Summary - Add `onConsoleLog` handler in vitest config to suppress all console output from tests (retry logs, settings errors, processor warnings) - Set `NODE_OPTIONS=--no-deprecation` to suppress punycode `DEP0040` warnings from vitest worker processes - Set `VITE_CJS_IGNORE_WARNING=true` to suppress Vite CJS API deprecation warning #skip-bugbot ## Test plan - Run `npm test` and verify clean output with no stderr noise - Verify all 810 tests still pass
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2665" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> --------- Co-authored-by:Claude Opus 4.6 <noreply@anthropic.com>
-
由 Will Chen 提交于
## Summary - Implements a new swarm planning skill that enables Product Manager, UX Designer, and Engineering Lead agents to collaboratively debate an idea - Agents analyze from their expert perspectives, debate trade-offs, and identify ambiguities - Produces a comprehensive plan document with problem statement, scope, user stories, UX design, technical design, and implementation roadmap ## Test plan - The skill can be invoked using `/dyad:swarm-to-plan <idea>` - It should spawn a planning team with three specialized agents - Agents should analyze the idea, debate assumptions, and compile a final plan to `plans/<plan-name>.md` - Verify the planning team shuts down gracefully after completion - Check that the generated plan includes all required sections from both PM, UX, and Engineering perspectives
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2670" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> -
由 Will Chen 提交于
## Summary - Add new Playwright debug skill to help developers debug E2E test failures - Provide instructions for using manual `page.screenshot()` calls to capture test state - Update AGENTS.md with guidance on when and how to use the skill ## Test plan - Review the skill documentation to ensure it provides clear instructions for: - Identifying which test to debug - Adding debug screenshots to test code - Viewing and analyzing captured screenshots - Cleaning up debug artifacts - The skill can be invoked with `/dyad:debug-with-playwright` when debugging E2E test failures
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2671" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> -
由 Will Chen 提交于
#skip-bb <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2669" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end -->
-
由 wwwillchen-bot 提交于
## Summary - Updates E2E test snapshot for security review to use `[[SYSTEM_MESSAGE]]` placeholder instead of the full system message content - Makes the test more stable and easier to maintain as system messages evolve ## Test plan - [x] Lint passes - [x] All unit tests pass (810 tests) - Snapshot update verified by running the E2E test that produced it
🤖 Generated with [Claude Code](https://claude.com/claude-code) #skip-bugbot <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2668" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> Co-authored-by:Will Chen <willchen90@gmail.com> Co-authored-by:
Claude Opus 4.5 <noreply@anthropic.com>
-
由 Will Chen 提交于
## Summary - Update nightly-runner-cleanup.yml to add ci3 runner to the cleanup matrix - Enhance ci-cleanup-macos.sh with improved disk space management for host-level caches - Add support for nightly cleanup mode with better handling of Homebrew and Xcode caches ## Test plan - Manually trigger the nightly-runner-cleanup workflow to verify cleanup steps complete successfully - Verify that the ci3 runner is now included in the cleanup matrix - Confirm that old browser installations and npm cache are properly cleaned up
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2664" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Small CI-maintenance change limited to a scheduled workflow and cleanup script; main risk is masking unexpected deletion failures via non-fatal warnings. > > **Overview** > Extends the nightly self-hosted macOS runner cleanup workflow to also run on `ci3` via the job matrix. > > Hardens `scripts/ci-cleanup-macos.sh` by making npm cache (`~/.npm/_cacache`) and log (`~/.npm/_logs`) removal tolerate failures (e.g., files in use), emitting warnings instead of failing the cleanup run. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit c49b18aa7df6e6da179507d678c67cc5680505c5. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Include the ci3 runner in the nightly macOS cleanup and make npm cache/log removal more resilient to prevent flaky failures. - **Bug Fixes** - Added ci3 to the nightly-runner-cleanup matrix so all macOS hosts are cleaned. - Updated ci-cleanup-macos.sh to ignore permission/in-use errors when removing npm cache/logs and emit warnings instead. <sup>Written for commit c49b18aa7df6e6da179507d678c67cc5680505c5. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> --------- Co-authored-by:claude[bot] <41898282+claude[bot]@users.noreply.github.com> Co-authored-by:
Claude Opus 4.5 <noreply@anthropic.com>
-
由 Will Chen 提交于
#skip-bb <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2667" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end -->
-
由 Will Chen 提交于
## Summary - Migrates all 16 `.claude/commands/` to `.claude/skills/` with proper SKILL.md frontmatter (name + description) - Moves supporting scripts (sanitize_issue_markdown.py, tests, goldens) into the fix-issue skill folder - Updates path references in `.prettierignore`, `.oxfmtrc.json`, and `.claude/README.md` - Eliminates spurious golden file "commands" (e.g., `dyad:scripts:goldens:*`) that were being registered as slash commands ## Test plan - Verify all skills are listed with `/` autocomplete - Invoke a skill (e.g., `/dyad:lint`) and confirm it works #skip-bugbot
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2663" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Migrates all Claude Code commands to the new skills format with SKILL.md frontmatter and updates docs/paths. This standardizes slash usage (/dyad:<skill>), fixes autocomplete, and removes stray commands. - **Refactors** - Moved 16 items from .claude/commands to .claude/skills with name/description frontmatter. - Moved sanitize_issue_markdown.py, tests, and goldens under the fix-issue skill; updated path in SKILL.md. - Updated .claude/README.md to list skills and include fast-push, pr-screencast, feedback-to-issues, deflake-e2e-recent-commits, remember-learnings. - Updated .prettierignore and .oxfmtrc.json to new goldens path. - Removed golden file “commands” that were being registered as slash commands. - **Migration** - Use /dyad:<skill> instead of /dyad:<command>. - Verify skills show in autocomplete and run one (e.g., /dyad:lint). - Update any local scripts from .claude/commands/... to .claude/skills/.... <sup>Written for commit 31ae67f994cb704c223428e66455665ffd2454ff. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> Co-authored-by:Claude Opus 4.6 <noreply@anthropic.com>
-
由 Will Chen 提交于
## Summary - Replace unreliable winget installation with NuGet installation for Azure Trusted Signing tools - Simplify DLL discovery logic to use direct NuGet installation path - Makes Windows signing step more robust and reliable ## Test plan - Verify that the release workflow still builds successfully on Windows runners - Confirm Azure Trusted Signing client is installed and used correctly
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2661" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Switch the Windows release workflow to use NuGet for Azure Trusted Signing and simplify DLL discovery to the NuGet install path. This reduces CI flakiness and makes the signing step more reliable. - **Refactors** - Install Microsoft.Trusted.Signing.Client 1.0.95 to $RUNNER_TEMP\TrustedSigning from nuget.org. - Find Azure.CodeSigning.Dlib.dll (x64) by scanning the install directory only. <sup>Written for commit 0dd327769005b7890b31d782bf07f66a4912406b. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Workflow-only change that alters how Windows signing dependencies are installed and discovered; risk is limited to potential release pipeline breakage if the NuGet package layout/version changes. > > **Overview** > Improves the Windows portion of the release workflow by installing Azure Trusted Signing tooling via `nuget` (pinned to `Microsoft.Trusted.Signing.Client` `1.0.95`) instead of `winget`. > > Simplifies DLL discovery by searching only within the NuGet install directory for the x64 `Azure.CodeSigning.Dlib.dll`, then exporting `AZURE_CODE_SIGNING_DLIB` for subsequent signing steps. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 0dd327769005b7890b31d782bf07f66a4912406b. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> Co-authored-by:Claude Opus 4.6 <noreply@anthropic.com>
-
由 Will Chen 提交于
## Summary - Windows CI builds intermittently fail during `npm ci` with `EBUSY`/`EPERM` errors caused by file locking (antivirus, indexing services) - Added a 3-attempt retry loop with 10s delay and `node_modules` cleanup between attempts - Applied to both the `build` and `e2e-tests` jobs in the CI workflow ## Test plan - [x] Lint passes - [x] Type checks pass - [x] Unit tests pass (810/810) - Verify the next Windows CI run completes the `npm ci` step successfully #skip-bugbot
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Add retry logic to npm ci on Windows runners to handle intermittent EBUSY/EPERM file locks. This stabilizes the build and e2e-tests jobs. - **Bug Fixes** - Retry npm ci up to 3 times with a 10s delay. - Clean node_modules between attempts. - Implemented via scripts/npm-ci-retry.sh and applied to both build and e2e-tests jobs. <sup>Written for commit 79bc6ede3b90db4068369e59853144170547aaf0. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2657" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> --------- Co-authored-by:Claude Opus 4.6 <noreply@anthropic.com>
-
由 Will Chen 提交于
#skip-bb <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2660" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end -->
-
由 Will Chen 提交于
## Summary - Updated ChatInputControls component with new functionality - Enhanced settings page with improved search indexing - Updated schema definitions with new type support - Updated chat stream handlers for better data processing - Added comprehensive e2e tests for Settings component ## Test plan - Run `npm run test` to verify all unit tests pass - Run `npm run e2e` to verify end-to-end tests pass - Manually test Settings page UI and functionality - Verify schema changes don't break existing data handling
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2659" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Medium Risk** > Changes the conditions under which MCP tools can be surfaced and executed in chat streaming, so mis-gating could unintentionally disable or enable tool execution paths; scope is limited to a new opt-in experiment flag. > > **Overview** > Introduces a new experimental setting, `enableMcpServersForBuildMode`, to gate MCP server usage outside Local Agent mode. > > When disabled, the MCP tools UI (`ChatInputControls`) and the MCP agent execution path in `chat_stream_handlers` no longer activate in Build mode; enabling the toggle re-enables MCP tools for Build/Agent behavior. > > Adds the toggle to the Settings "Experiments" section, indexes it for settings search, extends the settings schema to persist it, and updates MCP e2e tests/page objects to explicitly enable the experiment before exercising MCP tool calls. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 0b08d3da18e90fda6fc959a94e143df64249ad27. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Adds an experiment toggle to allow MCP servers in Build mode and gates MCP tool usage behind it across the chat UI and stream handlers. Updates Settings and search to surface the toggle, and updates e2e tests to cover the new flow. - **New Features** - Settings: new “Enable MCP servers for Build mode” toggle under Experiments; added to search. - Chat: ChatInputControls and chat_stream_handlers now require this flag; Agent mode remains supported; Build mode also needs at least one enabled MCP server. - Schema: adds enableMcpServersForBuildMode to UserSettingsSchema. - Tests: Playwright specs enable the toggle before MCP tests; new Settings page-object helper. - **Migration** - To use MCP in Build mode, enable Settings → Experiments → “Enable MCP servers for Build mode”. <sup>Written for commit 0b08d3da18e90fda6fc959a94e143df64249ad27. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> --------- Co-authored-by:Claude Opus 4.6 <noreply@anthropic.com>
-
由 Will Chen 提交于
## Summary - Updated node-version to node-version-file: package.json in release workflow - Allows the CI build to use Node >= 24 as required by project dependencies - Fixes the automated release build process ## Test plan - The release workflow will be tested on the next release trigger - Verify that the workflow can access the Node version from package.json
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2658" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Fixes failing release CI by reading the Node version from package.json instead of hardcoding v20, so the workflow uses Node 24+ and the release build runs again. - **Bug Fixes** - Switched actions/setup-node to node-version-file: package.json in two release jobs. - Ensures CI uses the repo’s Node version (24+) and unblocks automated releases. <sup>Written for commit f062289452ced8dbb62b47530ad0e98299d80a10. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Workflow-only change that adjusts runtime version selection; low risk outside of potential CI environment differences if `package.json` engines is misconfigured. > > **Overview** > Updates the GitHub Release workflow to source the Node version from `package.json` (via `node-version-file`) instead of hardcoding `node-version: 20` for both the build matrix and the `verify-assets` job. > > This aligns release CI with the repo’s declared Node engine requirement and avoids failures when dependencies require a newer Node than the previously pinned value. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit f062289452ced8dbb62b47530ad0e98299d80a10. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> Co-authored-by:Claude Opus 4.6 <noreply@anthropic.com>
-
由 Will Chen 提交于
#skip-bb <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2656" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end -->
-
由 wwwillchen-bot 提交于
## Summary - When a local agent ends its turn with incomplete todos (pending or in_progress), a reminder message is now injected telling it to continue and complete the remaining tasks - This only happens once per turn to avoid infinite loops - Added `hasIncompleteTodos()` and `buildTodoReminderMessage()` helpers to prepare_step_utils.ts - Added `TodoReminderState` to track whether a reminder has already been sent this turn Fixes #2600 ## Test plan - Unit tests added for: - `hasIncompleteTodos()` - correctly detects pending/in_progress todos - `buildTodoReminderMessage()` - builds proper reminder message listing incomplete todos - `prepareStepMessages()` with todoContext: - Injects reminder when agent finishes with incomplete todos - Does not inject reminder when already reminded this turn - Does not inject reminder when all todos are completed - Does not inject reminder when agent has pending tool calls - Combines reminder with existing injected messages🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2601" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Ensures the local agent completes remaining todos before ending a turn by running a one-time outer-loop follow-up pass that adds a reminder. The reminder is not persisted. Meets #2600. - **New Features** - Outer-loop detection via shouldRunTodoFollowUpPass: runs one follow-up pass when the final step has no tool calls and incomplete todos remain; skips in read-only and plan modes. - Helpers hasIncompleteTodos() and buildTodoReminderMessage(); multi-pass E2E fixture and test; fake LLM server scans all user messages and counts todo reminders to drive passes. - **Refactors** - Removed inner-loop reminder injection from prepareStepMessages; tests cleaned up. - Restructured local_agent_handler into a controlled pass loop with createdAt guards, baseMessageHistoryCount and compaction state reset each pass, AI messages persisted across passes, and synthetic todo reminders excluded from aiMessagesJson; updated compaction test to include toolCalls in mock steps. <sup>Written for commit 70b9c5a6595c5b024d25665e785d93dd77a3076f. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> --------- Co-authored-by:Will Chen <willchen90@gmail.com> Co-authored-by:
Claude Opus 4.5 <noreply@anthropic.com>
-
- 12 2月, 2026 19 次提交
-
-
由 wwwillchen-bot 提交于
## Summary - Updates both `pr-push.md` and `fast-push.md` commands to automatically remove the `needs-human:review-issue` label after pushing - This label indicates the issue needed human review before work started, which is now complete once the PR is pushed ## Test plan - Run `/dyad:pr-push` on a PR with the `needs-human:review-issue` label and verify it gets removed - Run `/dyad:fast-push` and verify the same behavior
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2654" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Automatically removes the needs-human:review-issue label after pushing in pr-push and fast-push, so issues reflect that human review is complete. Keeps PRs and labels in sync without manual cleanup. - **New Features** - pr-push and fast-push run gh pr edit --remove-label "needs-human:review-issue" after push, gracefully handling cases where the label doesn’t exist. <sup>Written for commit 5d9bd27c187502459b83bfde12b960a0f89c5cd5. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> Co-authored-by:Will Chen <willchen90@gmail.com> Co-authored-by:
Claude Opus 4.5 <noreply@anthropic.com>
-
由 Will Chen 提交于
## Summary - Add `$(cat ...)` as a safe command substitution pattern in the gh permission hook, allowing commands like `gh api graphql -f query="$(cat /tmp/query.graphql)" > /tmp/output.json` to passthrough instead of being blocked - Add safe pipe and redirect handling to the python permission hook, allowing pytest commands with `2>&1 | tail` output formatting patterns - Move `| cat` from blocked to allowed in python hook tests since `cat` is a safe read-only command #skip-bugbot ## Test plan - [x] All gh permission hook tests pass (572 good commands, 406 bad commands) - [x] All python permission hook tests pass (good, bad, passthrough, security-blocked) - [x] npm test passes (803 tests)
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2653" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Relaxed shell injection checks in GH and Python permission hooks to allow common read-only patterns like $(cat ...), pipes to safe tools, and simple redirects. This reduces false blocks while keeping unsafe substitutions and pipelines guarded. - **Bug Fixes** - GH hook: treat $(cat ...) as a safe command substitution and neutralize it before checks. - Python hook: allow pipes to common text tools (e.g., tail, grep, cat) and redirects like 2>&1 and >/dev/null. - Tests: move python `| cat` to allowed; add cases for gh `$(cat ...)` and pytest `2>&1 | tail`. <sup>Written for commit b1695c6e09bdc58288197ae2ab79745947fcca59. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> Co-authored-by:Claude Opus 4.6 <noreply@anthropic.com>
-
由 wwwillchen-bot 提交于
## Summary - Adds a GitHub Actions workflow that reviews Claude rules files (AGENTS.md and .mcp.json) on pull requests - The workflow checks for proper formatting and content in configuration files - Helps maintain quality of agent configuration and rules ## Test plan - PR review workflow will run on this PR - Verify the workflow file is valid YAML and GitHub Actions syntax #skip-bugbot
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2652" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Adds a weekly workflow that audits AGENTS.md and the rules/ directory and opens a single issue with keep/update/remove recommendations and proposed gaps. Keeps our agent guidance current and trims low‑value rules. - **New Features** - Adds .github/workflows/claude-rules-review.yml running Mondays at 13:00 UTC and via manual trigger. - Uses anthropics/claude-code-action@v1 in direct mode with restricted tools and a 30‑minute timeout (model set via claude_args). - Closes any prior agent-review issues before creating a new summary; ensures documentation and agent-review labels exist. - Requires CLAUDE_CODE_OAUTH_TOKEN; uses GITHUB_TOKEN; shallow clone to last 200 commits; minimal permissions (contents: read, issues: write). <sup>Written for commit 820abfb2c8cc30ece2743ac71f021e371d250c23. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> --------- Co-authored-by:Will Chen <willchen90@gmail.com> Co-authored-by:
Claude Opus 4.5 <noreply@anthropic.com> Co-authored-by:
claude[bot] <41898282+claude[bot]@users.noreply.github.com> Co-authored-by:
cubic-dev-ai[bot] <191113872+cubic-dev-ai[bot]@users.noreply.github.com>
-
由 Will Chen 提交于
## Summary - Extend the nightly disk cleanup workflow to run on both ci1 and ci2 self-hosted macOS runners - Update the guard step to accept ci2 as a valid runner name ## Test plan - Verify the workflow file syntax is correct - The nightly cleanup will run on both ci1 and ci2 runners
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2645" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Extend the nightly disk cleanup to run on both ci1 and ci2 using a strategy matrix with fail-fast: false so each runner executes independently. Removed the guard step and related conditions, and updated comments to reflect both runners. <sup>Written for commit d1a2a3d4d8d1392960aabf536afb152a9db89698. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Low risk CI workflow change that only affects scheduled cleanup execution on self-hosted runners; main risk is increased load or unexpected cleanup behavior now running on `ci2` as well. > > **Overview** > Expands the nightly macOS runner disk cleanup workflow to run on both self-hosted runners `ci1` and `ci2` by switching the job to a matrix strategy and targeting `runs-on: ${{ matrix.runner }}`. > > Removes the previous runner-name guard and conditional step execution so the cleanup script always runs for each matrix runner invocation. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit d1a2a3d4d8d1392960aabf536afb152a9db89698. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> --------- Co-authored-by:Claude Opus 4.5 <noreply@anthropic.com> Co-authored-by:
claude[bot] <41898282+claude[bot]@users.noreply.github.com>
-
由 wwwillchen-bot 提交于
## Summary - Store Dyad Pro status in localStorage for telemetry access - Sample 10% of events for non-Pro users (error events always sent) - Pro users continue to send all events ## Test plan - Verify telemetry events are sampled correctly for non-Pro users - Confirm error events are always sent regardless of Pro status - Test Pro users still send all events
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2642" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Adds telemetry sampling for non‑Pro users to reduce event volume while keeping error visibility. Stores Dyad Pro status in localStorage; Pro users keep sending all events. - **New Features** - Save Dyad Pro status in localStorage (dyadProStatus) and expose isDyadProUser(). - Sample PostHog events for non‑Pro: send 10% of non‑error events; always send errors. - No change for Pro users; all events continue to be sent. <sup>Written for commit 685922c99bc83c6c1d3c6e58a99761037894bc2b. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> Co-authored-by:Will Chen <willchen90@gmail.com> Co-authored-by:
Claude Opus 4.5 <noreply@anthropic.com>
-
由 Mohamed Aziz Mejri 提交于
<!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2561" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Refreshes the chat input UI for a cleaner look and consistent controls. Adds per‑mode icons and clearer visual states across inputs, pickers, and actions. - **UI Improvements** - Unified control sizing (h-7, text-xs, rounded-lg); vertical align center; input row aligns to end; removed spacer divs. - Borderless triggers for Mode/Model/Tools/Pro with subtle hover; smaller icons; muted “Model:”; mode chips show icons for all modes; Ask/Plan get color accents. - Input boxes rounded-2xl with focus-within ring; home input gets hover border; context banner rounded-t-2xl. - Send/Cancel use text-color hovers with clearer disabled; Cancel turns destructive on hover; Auxiliary Actions is circular with gentle hover scale. - Increased input padding and 15px text with repositioned placeholder; trimmed outer and MessagesList padding for tighter alignment; home send icon switched to SendHorizontal. <sup>Written for commit 301875efc229e998e9ba52b2eeeac057bd017b16. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> --------- Co-authored-by:
Will Chen <willchen90@gmail.com>
-
由 Will Chen 提交于
## Summary - Fix scroll behavior when switching between chats with existing messages - Use double `requestAnimationFrame` to wait for Virtuoso to render before scrolling to bottom when switching chats - Distinguish between chat switches and new message sends to handle scrolling appropriately - Avoid premature scrolling when switching to chats where messages haven't been fetched yet ## Test plan 1. Open Dyad and start a chat with some messages 2. Start another chat with messages 3. Switch between chats and verify scroll position stays at the bottom 4. Send a new message and verify it scrolls to show the new message
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2643" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Fixes chat auto-scroll to keep the view anchored at the bottom when switching chats or sending messages. Prevents flicker and jumps before messages render. - **Bug Fixes** - Distinguish chat switches vs new sends; adjust scroll timing accordingly. - On chat switch with existing messages, wait for Virtuoso to render (double requestAnimationFrame), then scroll to bottom instantly. - On new message send, wait for the placeholder to render (double requestAnimationFrame) before scrolling; skip auto-scroll when switching to a chat with no messages yet. <sup>Written for commit 8814a601a9615b60efbfbff0aff5b7150a75fa63. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Scoped to `ChatPanel` scroll timing logic; risk is limited to possible UI regressions (missed/extra scroll) when switching chats or starting streams. > > **Overview** > Fixes `ChatPanel` auto-scroll behavior by **distinguishing chat switches from new stream starts** and delaying the scroll until after Virtuoso has rendered. > > On chat switch, it now scrolls to bottom *only after messages exist* (avoiding premature scroll before fetch/render) and uses a double `requestAnimationFrame` with `instant` scrolling; on new message sends (`streamCount` changes), it similarly waits for render before scrolling with the default behavior. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 8814a601a9615b60efbfbff0aff5b7150a75fa63. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> --------- Co-authored-by:Claude Opus 4.5 <noreply@anthropic.com>
-
由 wwwillchen-bot 提交于
## Summary - Remove the unused `ChatActivity.tsx` component - Clean up all references including i18n translations (en, pt-BR, zh-CN) - Update e2e test helpers and concurrent chat spec to remove ChatActivity-related code ## Test plan - [x] Lint checks pass - [x] TypeScript compilation succeeds - [x] All 33 test files pass
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2648" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Removed the unused ChatActivity component and the bell entry point. Updated tests and i18n; e2e now targets the chat tabs “in progress” indicator instead of the activity list. - **Refactors** - Deleted ChatActivity.tsx and removed ChatActivityButton from TitleBar and ActionHeader. - Removed related i18n keys in en, pt-BR, and zh-CN. - Dropped e2e helpers for the activity list and updated concurrent_chat.spec to select the “Chat in progress” tab. <sup>Written for commit b005943bc7b87c8dfe6375504fbbbca088b7aabb. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> --------- Co-authored-by:Will Chen <willchen90@gmail.com> Co-authored-by:
Claude Opus 4.5 <noreply@anthropic.com>
-
由 wwwillchen-bot 提交于
## Summary - Changed `navigateToRoute()` to use `postMessage` + `location.replace()` pattern instead of direct `location.href` assignment - This matches the approach used by back/forward navigation buttons and provides smooth navigation without the black screen flicker and sidebar state reset - Also added proper `currentIframeUrlRef` and `preservedUrls` updates for HMR remount consistency Fixes #2428 ## Test plan - [x] Build succeeds - [x] Unit tests pass (784 tests) - [x] E2E tests pass for preview navigation (`npm run e2e -- --grep "preview navigation"`) - Manual testing: Open a multi-page app, use the route dropdown to navigate between routes - should no longer see black screen flicker or sidebar collapse
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2610" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Fixes #2428: removes black screen flicker and sidebar collapse when selecting routes from the preview dropdown. Navigation now uses postMessage + location.replace and keeps the iframe src stable across SPA navigation and HMR. - **Bug Fixes** - Use postMessage + location.replace to navigate; prevents flicker and preserves sidebar state. - Freeze iframe src with useMemo and same-origin check so SPA nav/HMR don’t reset it; added e2e test to verify src remains unchanged. - Sync navigation state: history, canGoBack/canGoForward, currentIframeUrlRef, and per-app preservedUrls (clears on root). <sup>Written for commit 1bdd7635ea432c564fe77fea57062df1a9d2f56d. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> --------- Co-authored-by:Will Chen <willchen90@gmail.com> Co-authored-by:
Claude Opus 4.5 <noreply@anthropic.com>
-
由 Mohamed Aziz Mejri 提交于
<!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2563" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Show a dialog after dragging or pasting files into chat to choose how to attach them—as chat context or upload to codebase. This makes intent explicit and blocks submit, drop, and paste until you choose. - **New Features** - Added FileAttachmentTypeDialog; integrated in ChatInput and HomeChatInput. i18n (en, pt-BR, zh-CN) with singular/plural titles and descriptions. - Updated useAttachments with pendingFiles and confirm/cancel. Drag/paste set pendingFiles; prevent attaching while pending; clearAttachments also clears pendingFiles; submit blocked when dialog is open. - Fixed e2e to select “Attach file as chat context”; dialog buttons use type="button" with focus-visible ring. - **Refactors** - confirmPendingFiles reuses addAttachments to deduplicate logic. <sup>Written for commit b625847b5ed5f82bea4616db27b87b16b5b33613. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> --------- Co-authored-by:
Claude Opus 4.6 <noreply@anthropic.com>
-
由 Will Chen 提交于
## Summary - Extends the `deflake-e2e-recent-commits` command to also gather flaky tests from open PRs authored by `wwwillchen` and `wwwillchen-bot` - Parses Playwright Test Results comments on these PRs to extract flaky test names - Provides more comprehensive coverage for deflaking efforts by combining main branch CI runs with PR-reported flakes ## Test plan - Run `/dyad:deflake-e2e-recent-commits` and verify it now scans both main branch CI runs AND open PRs by the specified authors - Verify flaky tests from PR comments are correctly parsed and added to the deflake list
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2647" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Extends deflake-e2e-recent-commits to also scan open PRs by wwwillchen and wwwillchen-bot for Playwright-reported flaky tests. This broadens coverage beyond main-branch CI and improves deflaking accuracy. - **New Features** - Lists recent open PRs by wwwillchen and wwwillchen-bot. - Parses the latest “Playwright Test Results” bot comment to extract flaky test titles. - Merges PR-derived flakes with main-branch results, de-duplicates, and notes PR sources in the summary. - Updates no-results message to include PRs (“recent commits or PRs”). <sup>Written for commit 32766d69227eb2454f45899e5784021161765019. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Documentation-only change that broadens the data sources described for collecting flaky tests; no runtime or production code is modified. > > **Overview** > Extends the `.claude` command `deflake-e2e-recent-commits` to **collect flaky Playwright tests from two sources**: recent `main` CI `html-report` artifacts *and* the latest “Playwright Test Results” bot comment on recent open PRs authored by `wwwillchen`/`wwwillchen-bot`. > > Updates the instructions to include the PR scanning/parsing workflow, to attribute flakes by source in the final report, and to change the no-flakes message to cover “recent commits or PRs.” > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 32766d69227eb2454f45899e5784021161765019. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> Co-authored-by:Claude Opus 4.5 <noreply@anthropic.com>
-
由 wwwillchen-bot 提交于
## Summary - Add error dialog handling in capacitor.spec.ts to gracefully dismiss sync errors that may occur in E2E environment due to missing CocoaPods/Xcode - Improve timing and wait logic in select_component.spec.ts for Next.js apps which take longer to compile and start the dev server - Update snapshot to use placeholder for system message instead of hardcoded content ## Test plan - Run `npm run test:e2e -- --grep "capacitor"` to verify the Capacitor test improvements - Run `npm run test:e2e -- --grep "select component next.js"` to verify the Next.js component selection test improvements - Verify that tests pass consistently without flakiness
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2646" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Stabilizes E2E tests by handling Capacitor sync errors and improving Next.js component selection timing to reduce flakiness. Also updates the snapshot to use a system message placeholder. - **Bug Fixes** - Capacitor: wait for sync completion and dismiss error dialog when CocoaPods/Xcode are missing. - Next.js: wait for preview iframe and heading visibility; add retry with toPass() for component selection. - Snapshot: replace hardcoded system message with [[SYSTEM_MESSAGE]] placeholder. <sup>Written for commit 0e33279d8d18c124ca11e95818ce76b8d9b7b4de. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> Co-authored-by:Will Chen <willchen90@gmail.com> Co-authored-by:
Claude Opus 4.5 <noreply@anthropic.com>
-
由 Will Chen 提交于
## Summary - Replace generic review prompts with three distinct reviewer personas (correctness, code-health, UX) - Each reviewer has a specialized focus area for more targeted feedback - Update workflow trigger to include skills directory changes ## Test plan - Verify the PR review workflow triggers correctly on PR changes - Confirm each reviewer persona produces relevant, focused feedback
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2644" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Refactors multi-pr-review to use three specialized reviewer personas—Correctness, Code Health, and UX—for more focused, actionable PR feedback. Updates the GitHub workflow to call the new skill. - **Refactors** - Added persona templates: correctness-reviewer.md, code-health-reviewer.md, ux-reviewer.md. - Updated SKILL.md with persona roles, severity guidance, and new references. - Removed old generic prompts: review_prompt_default.md and review_prompt_code_health.md. - Switched workflow command to /dyad:multi-pr-review. <sup>Written for commit f761c88cec8a91be810ca55f66db39b5b7acfa4e. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Medium Risk** > Changes affect the automated PR review pipeline and remove prompt files that the orchestrator may still reference, which could break CI-based reviews if not updated in lockstep. > > **Overview** > Updates the `dyad:multi-pr-review` documentation/spec to describe **three specialized sub-agent personas** (Correctness, Code Health, UX) and expands severity guidance to explicitly include UX blockers/degradations. > > Adds new persona reference docs (`correctness-reviewer.md`, `code-health-reviewer.md`, `ux-reviewer.md`) and removes the prior generic prompt templates (`review_prompt_default.md`, `review_prompt_code_health.md`). The PR review workflow is also updated to run `/dyad:multi-pr-review` instead of the previous command. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit f761c88cec8a91be810ca55f66db39b5b7acfa4e. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> Co-authored-by:Claude Opus 4.5 <noreply@anthropic.com>
-
由 Will Chen 提交于
## Summary - Extends the existing `gh pr` shell injection exemption to also cover `gh issue` commands in the permission hook - `gh issue create/comment/edit` frequently contain markdown in `--body` with backticks, pipes, `**bold**`, etc. that were incorrectly flagged as injection attempts - Added 50+ test cases covering `gh issue` and `gh pr` commands with rich markdown body content ## Test plan - [x] All 977 hook permission tests pass (`python .claude/hooks/tests/test_gh_permission_hook.py`) - [x] `npm run fmt && npm run lint:fix && npm run ts` passes - [x] `npm test` passes (33/33 test files) #skip-bugbot
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2638" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Extend the shell injection exemption to gh issue commands to prevent false positives on markdown in --body (backticks, pipes, bold). Other gh commands remain protected by injection checks. - **Bug Fixes** - Exempted "gh issue ..." (and kept "gh pr ...") from shell injection checks in the permission hook. - Added 50+ tests for rich markdown bodies across gh issue and gh pr commands. - Moved six gh issue cases from bad_commands to good_commands to match the exemption. - Preserved injection checks for all other gh commands. <sup>Written for commit e168f9b81384568475cbcb81a92ef19a0963c7a6. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> Co-authored-by:Claude Opus 4.6 <noreply@anthropic.com>
-
由 Will Chen 提交于
## Summary Adds a comprehensive GitHub Actions workflow that automatically responds to pull requests with AI-powered code reviews and automated fixes. The workflow handles PR labeling, manages retry logic (up to 3 retries), and integrates with Claude Code for automated problem resolution. ## Test plan - Verify the workflow is properly formatted and no syntax errors exist - Check that the workflow triggers on the correct events (pull_request_target with labels and workflow_run) - Verify that only trusted maintainers (wwwillchen, wwwillchen-bot) can trigger the workflow - Test the retry logic by checking label transitions (cc:request → cc:pending → cc:request:N) - Confirm the workflow creates meaningful git commits through Claude Code - Verify proper cleanup of labels when max retries (4 total) are reached
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2641" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Small workflow change, but it affects automation that can push to PR branches; failure modes are limited to CI/labeling behavior. > > **Overview** > The `PR Review Responder` workflow now runs an additional Claude Code step after `/dyad:pr-fix` to push any local commits that may have been created but not pushed (via a new `/dyad:fast-push` invocation). > > This reduces cases where the workflow would proceed to “check if commits were pushed” and label transitions without the PR branch actually being updated. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 27a3059357b5f2511abc39d1a7c3aedccd92b38c. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Adds a GitHub Actions workflow that automatically reviews pull requests, manages labels/retry logic, and applies fixes via Claude Code. Adds a fallback step to push any unpushed commits to ensure downstream workflows run and PR labels update correctly. <sup>Written for commit 27a3059357b5f2511abc39d1a7c3aedccd92b38c. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> -
由 Will Chen 提交于
## Summary Adds a new `/feedback-to-issues` command that converts customer feedback (usually emails or support tickets) into discrete GitHub issues. The command: - Parses feedback to extract concrete, actionable issues - Searches for existing duplicate issues on GitHub - Proposes new issues for user approval - Creates approved issues - Drafts a professional reply email to the customer ## Features - **Duplicate detection**: Searches GitHub with multiple keyword variations - **User approval workflow**: Shows proposed issues for review before creating - **Draft reply email**: Generates a response email acknowledging all items This is a useful tool for managing customer feedback and support tickets at scale. <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2639" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Documentation-only addition of a new Claude command; no application/runtime code paths change. > > **Overview** > Adds a new Claude command spec, `/.claude/commands/dyad/feedback-to-issues.md`, to turn pasted (or file-based) customer feedback into actionable GitHub issues. > > The workflow explicitly guides parsing feedback into discrete items, searching `gh issue list` for potential duplicates, presenting *already filed* vs *proposed* issues for **user approval**, then creating approved issues via `gh issue create` and drafting a reply email linking the resulting issues. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 546a1ace575909596613fa3b5863421bbafe1594. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Adds a new /feedback-to-issues command that turns customer emails or tickets into actionable GitHub issues. It checks for duplicates, requires approval before creating issues, and drafts a reply email with links. - **New Features** - Parses feedback into discrete issues with clear titles, types, and priorities. - Searches GitHub for duplicates using multiple keyword variations (open and closed). - Presents a review report and creates only approved issues via gh. - Drafts a concise customer reply that references existing and newly created issues. <sup>Written for commit 546a1ace575909596613fa3b5863421bbafe1594. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. -->
-
由 Will Chen 提交于
## Summary - Update claude-pr-review.yml with latest configuration - Adjust settings for improved PR review automation ## Changes - Modified `.github/workflows/claude-pr-review.yml` with updated configuration ## Test plan - Workflow configuration has been validated by lint checks - No breaking changes to existing functionality
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2640" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Workflow-only change that limits job runtime; low risk beyond possibly timing out unusually long review runs. > > **Overview** > Adds a 30-minute `timeout-minutes` to the `claude-review` job in `.github/workflows/claude-pr-review.yml` to prevent the automated PR review workflow from running indefinitely. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit 75736d6b53a2104d2fe2a030dc0101aabc72f6c6. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Set a 30-minute timeout for the Claude PR Review GitHub Actions job to prevent runaway runs and improve reliability. Added timeout-minutes: 30 to .github/workflows/claude-pr-review.yml; no functional changes to triggers or permissions. <sup>Written for commit 75736d6b53a2104d2fe2a030dc0101aabc72f6c6. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> -
由 wwwillchen-bot 提交于
## Summary - Prefetch user budget on app initialization to ensure budget info is available immediately on load - Invalidate user budget query when Dyad Pro API key is saved to reflect updated subscription status immediately - Added documentation for React Query prefetch and invalidation patterns to AGENTS.md ## Test plan - Verify that user budget data loads on app startup - Test saving a Dyad Pro API key and confirm the budget information updates automatically - Check that the app continues to work correctly without a Dyad Pro subscription
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2632" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Prefetches the user budget on app load and invalidates the budget query after saving a Dyad Pro API key, so subscription status and budget update instantly. Adds React Query docs for prefetch and invalidation patterns. <sup>Written for commit 92b15f29cda8b9b19ba04d281c6e3774ea3928c8. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. --> Co-authored-by:Will Chen <willchen90@gmail.com> Co-authored-by:
Claude Opus 4.5 <noreply@anthropic.com>
-
由 Will Chen 提交于
## Summary - Update language model constants with latest model configurations and pricing - Add support for new model versions across multiple providers - Improve model descriptions and configuration parameters ## Test plan - Verify that the application loads with all model configurations - Test model selection in the UI - Confirm no TypeScript or lint errors
🤖 Generated with [Claude Code](https://claude.com/claude-code) <!-- devin-review-badge-begin --> --- <a href="https://app.devin.ai/review/dyad-sh/dyad/pull/2633" target="_blank"> <picture> <source media="(prefers-color-scheme: dark)" srcset="https://static.devin.ai/assets/gh-open-in-devin-review-dark.svg?v=1"> <img src="https://static.devin.ai/assets/gh-open-in-devin-review-light.svg?v=1" alt="Open with Devin"> </picture> </a> <!-- devin-review-badge-end --> <!-- CURSOR_SUMMARY --> --- > [!NOTE] > **Low Risk** > Primarily updates configuration/constants for model selection; main behavioral change is the expanded free-model name matching which could alter which OpenRouter models are used in fallback. > > **Overview** > Adds the new Anthropic model `claude-opus-4-6` (with updated token limits/pricing metadata) to the selectable model options. > > Refreshes OpenRouter model options by introducing a generic `openrouter/free` entry, adding `z-ai/glm-5`, tweaking `glm-4.7` description, and removing older explicit free-model entries and `moonshotai/kimi-k2-0905`. Updates `FREE_OPENROUTER_MODEL_NAMES` to treat both `:free` and `/free` model names as free, which changes which models can be selected in the auto free fallback. > > <sup>Written by [Cursor Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit d192033357d87355a9d414269a19a348c3faef6d. This will update automatically on new commits. Configure [here](https://cursor.com/dashboard?tab=bugbot).</sup> <!-- /CURSOR_SUMMARY --> <!-- This is an auto-generated description by cubic. --> --- ## Summary by cubic Updated language model constants with the latest configs and pricing to keep model selection current. Adds new models and improves free model detection for OpenRouter. - **New Features** - Added Claude Opus 4.6 (1M context, 32k max output). - Added OpenRouter “free” model and Z‑AI GLM 5; updated GLM 4.7 description. - Updated context windows, temperatures, and pricing; removed deprecated free entries and Kimi K2 0905. - **Bug Fixes** - Free model detection now supports names ending with “/free” (not just “:free”). <sup>Written for commit d192033357d87355a9d414269a19a348c3faef6d. Summary will update on new commits.</sup> <!-- End of auto-generated description by cubic. -->
-