You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The release contains a significant number of GOOD commits (88) with clear value, but is substantially mixed with SLOP (26) and administrative noise. While the GOOD commits offer tangible fixes, features, and improvements, the SLOP ratio (22.4%) and presence of administrative updates suggest a targeted approach. No critical breaking changes or dependency conflicts were identified that would outright prevent merging, but careful selection of commits is advised to maximize value while minimizing unnecessary churn.
Batch classification
Verdict
Count
Branch
🟢 GOOD
88
sync/upstream/v3.17.4-good
🟡 NEEDS_REVIEW
2
sync/upstream/v3.17.4-needs-review
🔴 SLOP
26
sync/upstream/v3.17.4-slop
🟢 Good batch
Classified as real value. Candidate for merge.
ad85e438fix: include user/project agents in task(subagent_type) resolution -- Adds concrete functionality with clear user value (user/project agent resolution) and includes meaningful new tests covering various scenarios.
2266f28ffix: correct merge order so project agents override user agents -- The commit fixes a specific behavior issue by changing the merge order of project and user agents, ensuring project agents override user agents with the same name, and includes a new test case to validate this behavior.
64408823fix: actually swap merge order so project is added before user -- The commit fixes a specific behavior issue by changing the merge order of project and user agents, ensuring project agents override user agents with the same name, and includes a new test case to validate this behavior.
5441c848refactor: use spyOn instead of vi.mock for agent loader mocks -- The commit replaces vi.mock with spyOn for agent loader mocks, which is a clear refactor with a specific, justified improvement for idiomatic use in bun:test as suggested in a PR review.
b0b19f30fix(grep): handle Windows drive-letter paths and CRLF in parseOutput -- Fixes concrete behavior issues on Windows by updating regex patterns to handle drive-letter paths and CRLF line endings, ensuring the grep tool works correctly across platforms.
e9b6bba1refactor(agent-display): centralize name normalization resolution -- Adds new test cases for name normalization and refactors the resolution logic to centralize it, improving maintainability and coverage of edge cases like legacy names and UI prefixes.
f95e7a6crefactor(openclaw): reuse shared gateway url validation -- The commit refactors the code to reuse a shared gateway URL validation function, reducing duplication and improving maintainability.
40411f32docs(agents): add AGENTS.md documentation for prometheus, hephaestus, sisyphus variants, and builtin-skills -- Added substantial documentation for multiple agents and built-in skills with clear structure and purpose, providing real user value.
21822ba2test(openclaw): cover shared gateway url validation -- Adds meaningful new test coverage for gateway URL validation scenarios, including HTTPS, local HTTP, and invalid URLs.
a2ae3de3refactor(openclaw): extract gateway url validator -- Extracted a reusable function validateGatewayUrl with clear utility, reducing code duplication.
69f1c9aefix(doctor): add timeouts to subprocess spawns to prevent exit code 137 -- Adds timeouts to subprocess spawns to prevent hanging and SIGKILL (exit code 137), introducing a spawnWithTimeout utility and overall doctor command timeout with clear functional benefit.
314e1a5efix(telemetry): enable GeoIP resolution for PostHog events -- Explicitly enables GeoIP resolution for PostHog events by setting disableGeoip to false, providing clear functional benefit for geographic analytics.
d21ed3f7fix(doctor): add timeouts to subprocess spawns to prevent exit code 137 -- Adds timeouts to subprocess spawns and overall doctor command to prevent hanging and SIGKILL (exit code 137), introducing a spawnWithTimeout utility with clear functional benefit.
565d3ffafix(ci): build darwin binaries on macOS runners to restore code signing -- The commit fixes a concrete compatibility issue with macOS Gatekeeper by adjusting the CI workflow to build darwin binaries natively on macOS runners and adds an explicit ad-hoc codesign step.
71b5ab67fix(ci): replace breaking codesign step with signature verification -- Replaces a breaking CI codesign step with signature verification, addressing a specific compatibility issue with macOS ad-hoc signatures.
16e7fa5bfix(ci): ad-hoc sign darwin binaries with plain codesign -- Explicitly signs Darwin binaries with 'codesign --sign -' to address a concrete CI compatibility issue with macOS runners.
470ed8b1fix(ci): remove existing signature before ad-hoc signing darwin binary -- The commit fixes a concrete compatibility issue with macOS Gatekeeper by removing an existing malformed signature before applying a fresh ad-hoc signature to Darwin binaries.
2d65896brefactor(shared): simplify normalize SDK null guards -- Simplified null checks in normalizeSDKResponse function from explicit null and undefined checks to a single '== null' check, improving code readability without changing behavior.
79a475d6refactor(background-agent): simplify loop detector null guard -- Simplified null checks in a function without changing behavior, improving code readability.
50df6f0dtest(claude-code-plugin-loader): cover plugin path nullish resolution -- Adds meaningful test coverage for plugin path resolution, including nullish value handling.
668bc8e8refactor(config-manager): simplify config parsing guards -- Simplifies config parsing logic by removing redundant conditional checks and adds meaningful test coverage for config file parsing scenarios.
a0d5131erefactor(plugin): remove dead chat params code -- Removes dead code (unused 'variant' parameter and associated logic) in chat-params.ts, simplifying the buildChatParamsInput function without affecting functionality.
21cad26cdocs(config): remove redundant schema default comments -- Removed redundant default value comments in schema definitions, improving code clarity without changing behavior.
4de02094test(background-agent): lock nullish loop detector behavior -- Adds meaningful test coverage for nullish input handling in loop detector and tool call signature creation.
141798efrefactor(background-agent): standardize loop detector null guards -- Simplified null checks in two functions without changing behavior, improving code readability.
3d2eb6e4fix(config): preserve explicit git_master overrides during merge -- Adds meaningful test coverage for preserving explicit git_master settings during merge, with clear behavior delta and concrete test cases.
dac0c99efeat: add Vercel AI Gateway as a provider -- Adds Vercel AI Gateway as a recognized provider with clear integration across multiple files and test updates, providing real user value.
aac17135test: add vercel provider transform tests -- Adds multiple meaningful tests for Vercel provider transformations with clear expected outputs, covering various model types and edge cases.
6f72ec75test: add integration tests for Vercel AI Gateway model resolution -- Adds integration tests for Vercel AI Gateway model resolution with clear test cases and expected snapshots, providing meaningful test coverage.
542dc890fix(provider): use gateway-specific model IDs for vercel transform -- Fixes behavior for Vercel AI Gateway by using correct model IDs without '-preview' suffix, with clear test case updates and logic adjustments in transform functions.
34e334eafeat(provider): add vercel to all gateway-supported fallback entries -- Adds vercel provider fallback entries with clear integration and test updates, providing real user value.
effae16crefactor(provider): replace per-model string replacements with generic regex -- Replaced multiple per-model string replacements with a generic regex, simplifying the code and making it more maintainable for future models.
680d05a2Prometheus: keep planner primary-only -- Changes Prometheus agent mode from 'all' to 'primary' across multiple tests and config builders, aligning with the stated goal of keeping the planner primary-only, with clear test updates and direct functional impact.
e3b5c2befix: address review feedback for Vercel AI Gateway PR -- Adds Vercel AI Gateway integration with CLI option, priority adjustment, and test updates, providing clear user value and functionality.
01a1b141fix(delegate-task): reject primary agents in task subagent resolution -- The commit fixes a concrete behavior issue by rejecting primary agents in task subagent resolution, introducing new functions with clear utility and direct functional impact.
3673b962docs(readme-ru): add telemetry and legal section translation -- Added meaningful Russian translations for telemetry and legal sections in README.ru.md, including privacy policy and terms of service links with clear user value.
23d0a0dcdocs(readme-zh-cn): add telemetry and legal section translation -- Added meaningful Chinese translations for telemetry and legal sections in README.zh-cn.md, including privacy policy and terms of service links with clear user value.
0ece66e3docs(readme-ja): add telemetry and legal section translation -- Added meaningful Japanese translations for telemetry and legal sections in README.ja.md, including privacy policy and terms of service links with clear user value.
12ed9501docs(readme-ko): add telemetry and legal section translation -- Added meaningful Korean translations for telemetry and legal sections in README.ko.md, including privacy policy and terms of service links with clear user value.
c91db2ebdocs(cli): add --vercel-ai-gateway flag documentation -- Added documentation for a new CLI flag with clear description and default value, providing direct user information.
e223ab20docs(configuration): add sisyphus-junior to agent list -- Added a new agent 'sisyphus-junior' to the list of available agents in the configuration documentation, providing clear user value by expanding the set of configurable options.
9342e55bdocs(installation): replace npx with bunx and update provider flags -- Updated installation docs to reflect 'bunx' usage and added relevant provider flags with clear user value.
05e54444docs(overview): update model-role claims and complete category list -- The commit updates the documentation to include a complete and detailed category list for model roles, enhancing user understanding and configuration clarity.
7c94ee45docs(agent-model-matching): sync fallback chains with current source -- Updated fallback chains in documentation to match current source, adding 'vercel' provider paths for clarity and consistency across agent and category configurations.
b57a7975docs(ollama): verify troubleshooting steps and issue links -- Updated troubleshooting steps with verified issue links and clear documentation improvements for user value.
bbbd1bd0docs(orchestration): update diagram model names and category tables -- The commit updates documentation with new, specific, and informative content (e.g., model fallback chains, category tables) that enhances user understanding and configuration clarity.
4ce1dde3CLI: isolate provider model id transforms from shared mocks -- Added meaningful tests for model ID transformations across multiple providers with clear expected outputs.
1b36194bfix(cli): treat missing session status as idle in run completion -- Added a test case to handle missing session status as idle, ensuring the CLI doesn't hang indefinitely, and updated the logic in pollForCompletion to return 'idle' when the session ID is missing from the statuses.
b5250555fix(telemetry): guard PostHog init failures -- Adds meaningful test coverage for PostHog init failure scenarios, ensuring no-op client behavior without throwing errors.
62c60ae9fix: numeric skill names, ultrawork missing run_in_background, ZWSP agent lookups -- The commit fixes concrete bugs with clear functional impact, including handling numeric skill names, adding a required parameter for ultrawork tasks, and stripping invisible characters for agent name comparisons.
76c5356atest(agent-config): add regression tests for agent merge priority order -- Adds meaningful regression tests for agent merge priority order with clear test cases covering various scenarios.
da91c535fix(call-omo-agent): address cubic review findings and add requirement-based tests -- The commit fixes concrete issues, adds meaningful tests, and improves functionality with clear value.
1e85a88dfix(agent-restrictions): restore EXPLORATION_AGENT_DENYLIST as default fallback for unknown agents -- Restores a default fallback for unknown agents, directly impacting functionality with clear value.
fd28f7e6feat(agents): add agent_definitions schema, eager path resolution, and JSON agent loader -- Adds agent_definitions schema, eager path resolution, and JSON agent loader with comprehensive test suite and clear functional impact.
5755a90cfeat(agents): add agent definitions file loader and opencode.json reader -- Adds agent definitions loader with comprehensive test coverage and clear functional value, including support for multiple file formats and logical loading precedence.
39bda91bfeat(agents): wire agent_definitions and opencode.json agents into precedence chain -- The commit integrates agent_definitions and opencode.json agents into the precedence chain with clear functional impact, new tests, and logical filtering logic.
3d0fb22cfix(call-omo-agent): adapt dev's model override tests to use createMockCtx factory -- The commit adapts existing tests to use a new factory function (createMockCtx) instead of a deprecated constant (mockCtx), ensuring test compatibility with the current codebase.
42445f51fix(agents): address cubic review findings on agent loader -- The commit addresses specific cubic review findings with concrete behavior fixes, including case-insensitive file extension handling, relative path resolution, and precedence logic adjustments, all backed by new tests.
cf4b2315fix(agents): replace 'in' with Object.hasOwn() for prototype-safe property checks -- Replaces 'in' with Object.hasOwn() for prototype-safe property checks in two occurrences, addressing a specific review finding to ensure accurate own-property checks on plain objects.
4c77045cfix(agents): use null-prototype accumulator to prevent proto pollution -- The commit fixes a concrete security issue by preventing proto pollution using a null-prototype accumulator, directly improving code safety with clear functional value.
ab11f2ebfix(tests): replace mock.module with spyOn to prevent test pollution -- Replaced mock.module() with spyOn() to prevent test pollution, ensuring proper restoration in afterEach()
47aa3025fix(cli): use getAgentRuntimeName for agent resolution in run command -- The commit fixes the agent resolution in the run command by using getAgentRuntimeName, which is a concrete behavior fix with clear necessity.
0764526afix(posthog): disable exception autocapture to stay within free tier -- Disables exception autocapture to stay within PostHog's free tier limit, preserving manual capture for critical errors.
a1842f2dfeat(tool-metadata): add shared metadata contract and bridge -- Adds a shared metadata contract and bridge with meaningful tests covering various scenarios, including round-trip persistence, case insensitivity, and edge cases like missing or blank call IDs.
80d3339cfeat(background-agent): add wait-for-task-session helper -- Adds a waitForTaskSessionID helper with comprehensive tests covering immediate resolution, polling, abort signals, timeouts, and terminal task statuses, providing clear functional value for task session management.
4da30057fix(plugin): harden metadata recovery and extraction -- The commit introduces meaningful functionality to extract session IDs from metadata, including legacy aliases, with clear tests and functional value.
c6a407c4refactor(tools): migrate producers to shared metadata bridge -- The commit refactors multiple files to migrate producers to a shared metadata bridge, replacing storeToolMetadata with publishToolMetadata and removing redundant resolveCallID logic, indicating a clear structural improvement with broad applicability.
Branch: sync/upstream/v3.17.4-good (local only; not on origin)
⚠️ 6 commit(s) failed to cherry-pick and are NOT in the draft PR.
🚫 Push to origin was rejected. GITHUB_TOKEN cannot create or modify files under .github/workflows/. The batch branch exists only in the workflow runner and was discarded. To merge this batch, run the analyzer with a PAT that has the workflow scope, or cherry-pick locally.
📝 4 commit(s) touch .github/workflows/ and require elevated permissions to sync: 565d3ff, 71b5ab6, 16e7fa5, 470ed8b
🟡 Review batch
Ambiguous. Human should review before merging.
cd8352c1refactor(claude-code-mcp-loader): simplify env expansion null guard -- Classification failed: No object generated: response did not match schema.
569addd3docs(provider): add comments to vercel transform logic -- Classification failed: No object generated: response did not match schema.
Branch: sync/upstream/v3.17.4-needs-review (on origin)
⚠️ 1 commit(s) failed to cherry-pick and are NOT in the draft PR.
🔴 Slop batch
Classified as AI slop or pointless churn. Do not merge as-is; rewrite intent if salvageable.
1e2e0019refactor(create-tools): keep result type internal -- The commit changes the visibility of CreateToolsResult from export to internal without any clear functional or architectural benefit, indicating minor visibility churn.
9d89bbb0refactor(commands): keep builtin command options internal -- Changes the visibility of LoadBuiltinCommandsOptions from export to internal without clear functional or architectural benefit
0c5cd3a1chore(agents): update all AGENTS.md generation dates to 2026-04-11 -- The commit updates generation dates in multiple AGENTS.md files without adding new information or functionality, indicating date metadata churn.
6713b30crefactor(hooks): drop no-op directory injector callbacks -- Removed no-op callbacks but introduced overly complex hook interfaces with unclear value
678011c7docs(contributing): update tool/hook counts and version references -- The commit updates counts and version references in the contributing docs without adding new substantive information.
cfd152addocs(features): update agent count and verify category list -- Added two new features ('review-work' and 'ai-slop-remover') with overly broad, vague descriptions lacking concrete behavior deltas or clear user value metrics.
0cb0a6c0docs(contributing): update tool/hook counts and version references -- Updated counts and version references in docs without adding new substantive information.
1bb59c3erelease: v3.17.2 -- This commit is a release version bump with no code changes, only updating package versions to 3.17.2 across multiple platform-specific binaries.
fe091ef2chore: bump version to 3.17.4 -- The commit only updates the version number in package.json with no functional changes.
c9350c67release: v3.17.4 -- The commit only updates version numbers in multiple package.json files with no functional changes, constituting administrative noise.
Branch: sync/upstream/v3.17.4-slop (on origin)
Slop verifier second-pass notes
1e2e001 CONFIRMED_SLOP (behavior delta: none): The commit changes the visibility of CreateToolsResult from export to internal without any clear functional or architectural benefit, indicating minor visibility churn. There is no concrete behavior delta, and the abstraction's change in visibility does not introduce new functionality or fix an issue. The commit message accurately reflects the change but does not justify its value. A senior engineer would likely question the necessity of this change without further context on planned usage or dependencies that require this internalization.
9d89bbb CONFIRMED_SLOP (behavior delta: none): The commit changes the visibility of LoadBuiltinCommandsOptions from export to internal without clear functional or architectural benefit. This is classified as minor_visibility_churn, and without a concrete benefit, it remains SLOP. No behavior delta is introduced as the interface's usage scope is merely reduced, not altered in functionality.
0c5cd3a CONFIRMED_SLOP (behavior delta: none): The commit updates generation dates in multiple AGENTS.md files without adding new functionality or information, indicating date metadata churn. This aligns with the 'date_count_metadata_churn' slop signal. No runtime code paths change, and the commit message accurately reflects the change but lacks measurable value. A senior engineer would likely not defend this as valuable in a code review focused on merge value for this fork.
e654cfb CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value. The first-pass reason correctly identifies it as cla_admin, a category excluded from being demoted to GOOD by the verification bias. There is no concrete behavior delta, and the change does not introduce any new functionality or fixes.
6713b30 CONFIRMED_SLOP (behavior delta: minor): The commit removes no-op callbacks but introduces overly complex hook interfaces without clear value. The changes do not alter runtime behavior significantly (behavior_delta: minor) and the complexity added does not have an obvious payoff or multiple callers. The commit message accurately reflects the removal of no-op code but does not justify the new complexity. A senior engineer would likely question the necessity of the introduced interfaces without a clear use case or future plan. The first-pass verdict correctly identified this as SLOP due to the workaround-heavy approach and unclear benefits.
6a0d515 CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value. The first-pass verdict correctly identifies it as cla_admin, a category excluded from being demoted to GOOD by the verification bias. There is no concrete behavior delta, and the change does not contribute to the functionality or quality of the codebase.
5b33cd2 CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value. This matches the 'cla_admin' category, which is explicitly non-GOOD by the verification bias rules.
c6deb6d CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value. This aligns with the 'cla_admin' category, which is explicitly non-GOOD by the verification bias rules. There is no behavior delta, no new functionality, and the change does not impact any runtime code path or user-visible output.
42df1c3 CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value. The first-pass reason correctly identifies it as cla_admin, a category excluded from being demoted to GOOD by the verification bias. No behavior delta exists as this change does not affect runtime code or user-visible output.
84f980c CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value. The first-pass reason correctly identifies this as cla_admin, a category excluded from being demoted to GOOD by the verification bias. There is no concrete behavior delta, and the change does not introduce any new functionality or improvements.
678011c CONFIRMED_SLOP (behavior delta: none): The commit updates counts and version references in the contributing docs without adding new substantive information, aligning with documented slop patterns for metadata churn. No runtime behavior changes, and the updates are purely bookkeeping. A senior engineer would likely question the value given the lack of new content or functionality.
cfd152a CONFIRMED_SLOP (behavior delta: none): The commit adds two new features with vague descriptions lacking concrete behavior deltas or clear user value metrics. The commit message's 'verify category list' is unmeasurable, and the changes expand existing docs without new facts. No runtime code changes occur, and the benefits are not quantifiable or directly observable.
0cb0a6c CONFIRMED_SLOP (behavior delta: none): The commit updates counts and version references in documentation without adding new substantive information. The changes are purely numerical and do not introduce new facts, functionality, or value beyond keeping existing content up-to-date, which aligns with the first-pass reason for SLOP. No behavior delta exists as it's a doc change.
3419bbc CONFIRMED_SLOP (behavior delta: none): The commit only updates a CLA signature in cla.json, which is administrative noise without code or product value. The first-pass reason correctly identifies it as cla_admin, a category excluded from being demoted to GOOD by the verification bias. No behavior delta exists as it doesn't alter runtime code or user-visible output.
1bb59c3 CONFIRMED_SLOP (behavior delta: none): The commit is a release version bump from 3.17.0 to 3.17.2 with no code changes, only updating version numbers in multiple package.json files. This matches the 'release_version_bump' slop pattern. There is no concrete behavior delta, as no runtime code paths or user-visible outputs are altered. The commit's sole purpose is administrative version updating, which is classified as SLOP by the given rubric.
571cf1f CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json without any code or product changes, aligning with documented slop patterns for administrative noise. No behavior delta exists as it doesn't affect runtime code or user-visible output. The commit message accurately reflects the change but lacks value beyond administrative tracking.
f98f7ec CONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json, which is administrative noise with no direct code or product value. The first-pass verdict correctly identifies it as SLOP based on the documented pattern for CLA signatures. No behavior delta exists as this change does not affect runtime code or user-visible output.
d97c067 CONFIRMED_SLOP (behavior delta: none): The commit only updates a CLA signature in cla.json without any code or product changes. This matches the 'cla_admin' category, which is not considered GOOD by default. There's no behavior delta, and the change is administrative noise.
eaf5ff2 CONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json, which is administrative noise with no direct code or product value. The first-pass verdict correctly identifies it as cla_admin, a category excluded from being demoted to GOOD by the verification bias rules. No behavior delta exists as this change does not affect runtime code or user-visible output.
f8f7ac1 CONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json, which is administrative noise with no direct code or product value. The first-pass verdict correctly identifies it as cla_admin, a category excluded from being demoted to GOOD by the verification bias rules.
54a94d4 CONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json with no code or product changes, aligning with documented slop patterns for administrative noise. No behavior delta exists as it's purely metadata. The first-pass verdict correctly identifies it as SLOP due to lacking merge value for the fork.
81a03fa CONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json, which is purely administrative with no impact on code functionality or product value. This aligns with the 'cla_admin' slop pattern, and there's no concrete behavior delta or value addition. The first-pass verdict correctly identifies it as SLOP.
9aeb87c CONFIRMED_SLOP (behavior delta: none): The commit only updates the version number in package.json with no functional changes, aligning with the 'release_version_bump' slop pattern. No behavior delta exists as it's purely administrative. A senior engineer would likely not defend this as valuable in a code review focused on functional merge value. The commit message accurately reflects the change but lacks measurable value beyond administrative tracking.
80e73f5 CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value. The first-pass reason correctly identifies it as cla_admin, a category excluded from being demoted to GOOD by the verification bias. No behavior delta exists as this change does not affect runtime code or user-visible output.
fe091ef CONFIRMED_SLOP (behavior delta: none): The commit only updates the version number in package.json with no functional changes, aligning with the 'release_version_bump' slop pattern. No behavior delta exists as it's purely administrative. A senior engineer would likely not defend this as valuable in a code review focused on functional merge value.
c9350c6 CONFIRMED_SLOP (behavior delta: none): The commit only updates version numbers in multiple package.json files with no functional changes, constituting administrative noise (release_version_bump). There is no concrete behavior delta, and the change is purely administrative.
Fork-sync impact assessment
Breaking changes
(none detected)
Dependency changes
(none)
Architecture drift
(none)
Hidden concerns
Minor refactor churn in some GOOD commits may require careful review for consistency.
Action items before merging
Manually review refactor commits for consistency with fork's coding standards.
Verify that doc updates align with fork's documentation guidelines.
Assess the necessity of each SLOP commit if considering broader inclusion.
Generated by upstream-analyzer workflow. This issue is advisory: no merges or publishes happen automatically. Phase 1: draft PRs only. The prompt lives in .github/prompts/ and is editable without touching workflow code.
🟡 Upstream cost-benefit: v3.17.0 → v3.17.4
Upstream:
code-yeongyu/oh-my-openagentRecommendation:
CHERRY_PICK(confidence: medium)Slop ratio: 22%
The release contains a significant number of GOOD commits (88) with clear value, but is substantially mixed with SLOP (26) and administrative noise. While the GOOD commits offer tangible fixes, features, and improvements, the SLOP ratio (22.4%) and presence of administrative updates suggest a targeted approach. No critical breaking changes or dependency conflicts were identified that would outright prevent merging, but careful selection of commits is advised to maximize value while minimizing unnecessary churn.
Batch classification
sync/upstream/v3.17.4-goodsync/upstream/v3.17.4-needs-reviewsync/upstream/v3.17.4-slop🟢 Good batch
Classified as real value. Candidate for merge.
ad85e438fix: include user/project agents in task(subagent_type) resolution -- Adds concrete functionality with clear user value (user/project agent resolution) and includes meaningful new tests covering various scenarios.2266f28ffix: correct merge order so project agents override user agents -- The commit fixes a specific behavior issue by changing the merge order of project and user agents, ensuring project agents override user agents with the same name, and includes a new test case to validate this behavior.64408823fix: actually swap merge order so project is added before user -- The commit fixes a specific behavior issue by changing the merge order of project and user agents, ensuring project agents override user agents with the same name, and includes a new test case to validate this behavior.5441c848refactor: use spyOn instead of vi.mock for agent loader mocks -- The commit replaces vi.mock with spyOn for agent loader mocks, which is a clear refactor with a specific, justified improvement for idiomatic use in bun:test as suggested in a PR review.b0b19f30fix(grep): handle Windows drive-letter paths and CRLF in parseOutput -- Fixes concrete behavior issues on Windows by updating regex patterns to handle drive-letter paths and CRLF line endings, ensuring the grep tool works correctly across platforms.e9b6bba1refactor(agent-display): centralize name normalization resolution -- Adds new test cases for name normalization and refactors the resolution logic to centralize it, improving maintainability and coverage of edge cases like legacy names and UI prefixes.f95e7a6crefactor(openclaw): reuse shared gateway url validation -- The commit refactors the code to reuse a shared gateway URL validation function, reducing duplication and improving maintainability.40411f32docs(agents): add AGENTS.md documentation for prometheus, hephaestus, sisyphus variants, and builtin-skills -- Added substantial documentation for multiple agents and built-in skills with clear structure and purpose, providing real user value.21822ba2test(openclaw): cover shared gateway url validation -- Adds meaningful new test coverage for gateway URL validation scenarios, including HTTPS, local HTTP, and invalid URLs.a2ae3de3refactor(openclaw): extract gateway url validator -- Extracted a reusable functionvalidateGatewayUrlwith clear utility, reducing code duplication.69f1c9aefix(doctor): add timeouts to subprocess spawns to prevent exit code 137 -- Adds timeouts to subprocess spawns to prevent hanging and SIGKILL (exit code 137), introducing a spawnWithTimeout utility and overall doctor command timeout with clear functional benefit.314e1a5efix(telemetry): enable GeoIP resolution for PostHog events -- Explicitly enables GeoIP resolution for PostHog events by setting disableGeoip to false, providing clear functional benefit for geographic analytics.d21ed3f7fix(doctor): add timeouts to subprocess spawns to prevent exit code 137 -- Adds timeouts to subprocess spawns and overall doctor command to prevent hanging and SIGKILL (exit code 137), introducing a spawnWithTimeout utility with clear functional benefit.565d3ffafix(ci): build darwin binaries on macOS runners to restore code signing -- The commit fixes a concrete compatibility issue with macOS Gatekeeper by adjusting the CI workflow to build darwin binaries natively on macOS runners and adds an explicit ad-hoc codesign step.71b5ab67fix(ci): replace breaking codesign step with signature verification -- Replaces a breaking CI codesign step with signature verification, addressing a specific compatibility issue with macOS ad-hoc signatures.16e7fa5bfix(ci): ad-hoc sign darwin binaries with plain codesign -- Explicitly signs Darwin binaries with 'codesign --sign -' to address a concrete CI compatibility issue with macOS runners.470ed8b1fix(ci): remove existing signature before ad-hoc signing darwin binary -- The commit fixes a concrete compatibility issue with macOS Gatekeeper by removing an existing malformed signature before applying a fresh ad-hoc signature to Darwin binaries.2d65896brefactor(shared): simplify normalize SDK null guards -- Simplified null checks in normalizeSDKResponse function from explicit null and undefined checks to a single '== null' check, improving code readability without changing behavior.79a475d6refactor(background-agent): simplify loop detector null guard -- Simplified null checks in a function without changing behavior, improving code readability.50df6f0dtest(claude-code-plugin-loader): cover plugin path nullish resolution -- Adds meaningful test coverage for plugin path resolution, including nullish value handling.668bc8e8refactor(config-manager): simplify config parsing guards -- Simplifies config parsing logic by removing redundant conditional checks and adds meaningful test coverage for config file parsing scenarios.a0d5131erefactor(plugin): remove dead chat params code -- Removes dead code (unused 'variant' parameter and associated logic) in chat-params.ts, simplifying the buildChatParamsInput function without affecting functionality.21cad26cdocs(config): remove redundant schema default comments -- Removed redundant default value comments in schema definitions, improving code clarity without changing behavior.4de02094test(background-agent): lock nullish loop detector behavior -- Adds meaningful test coverage for nullish input handling in loop detector and tool call signature creation.141798efrefactor(background-agent): standardize loop detector null guards -- Simplified null checks in two functions without changing behavior, improving code readability.e71c34acfix(migration): return true when canonical config write succeeds regardless of archive status ([Bug]: v3.15.1 still inconsistently loads/migrates plugin config after rename code-yeongyu/oh-my-openagent#3133) -- Fixes a concrete behavior issue where migration incorrectly returned false due to archive failure despite successful canonical config write.7accb53cfix(shared): handle Windows rename-over-existing in write-file-atomically ([Bug]: Using oh-my-opencode via bunx causes a new oh-my-openagent.json.bak.* backup file to be created on every OpenCode startup, even when no config changes were made. code-yeongyu/oh-my-openagent#3222) -- Fixes a concrete behavior issue on Windows by handling rename-over-existing errors with a fallback strategy, adding meaningful test coverage for atomic file writes.9ef133a8fix(migration): register parenthesized legacy agent aliases in AGENT_NAME_MAP ([Bug]: Header 'x-opencode-agent-name' has invalid value: 'Atlas (Plan Executor)' code-yeongyu/oh-my-openagent#3281) -- Adds concrete mappings for legacy agent aliases and tests for migration correctness, addressing a specific compatibility issue.a6e4f211fix(shared): normalize claude model IDs for anthropic provider (transformModelForProvider missing anthropic provider — dash-format model IDs rejected by OpenCode code-yeongyu/oh-my-openagent#3290) -- Fixes a concrete compatibility issue by normalizing model IDs for Anthropic API, with clear test expectation updates.4f02ace7fix(shared): guard maxOutputTokens <= 0 in model settings compatibility ([Bug]: Invalid argument for parameter maxOutputTokens: maxOutputTokens must be >= 1 code-yeongyu/oh-my-openagent#3305) -- Fixes a concrete behavior issue by adding a guard to ignore invalid maxOutputTokens metadata, preserving user-requested maxTokens with clear test coverage.c750781bfix(shared): avoid false-positive skill path resolution on npm scoped packages ([Bug]: Why my command run with error path? code-yeongyu/oh-my-openagent#2857) -- Fixes a concrete behavior issue by adjusting regex to exclude npm scoped packages, with clear test addition.d7b4bec5fix(hooks,tools): replace /plan example with prometheus delegation and rename code-review example to review-work (Error message references non-existent /plan command code-yeongyu/oh-my-openagent#2633, [Bug]: Skill tool schema uses non-existent "code-review" as example instead of actual "review-work" skill code-yeongyu/oh-my-openagent#3285, [Bug]: oh-my-openagent overrides built-in tools to bypass the permission system. code-yeongyu/oh-my-openagent#2873) -- The commit fixes concrete behavior issues by replacing a non-existent command with a valid Prometheus delegation and renaming a non-existent skill to an actual built-in one, with clear test additions.7e96af5ffix(atlas): exclude node_modules from verification git diff --stat ([Feature]: Ensure gitignore is created before starting implementation code-yeongyu/oh-my-openagent#3215) -- The commit fixes a concrete behavior issue by excluding node_modules from git diff verification, with clear test additions.2d8b4a9dfix(anthropic-effort): skip effort injection for github-copilot provider ([Bug]: code-yeongyu/oh-my-openagent#3270) -- Explicit guard added to skip effort injection for github-copilot provider, preserving native Anthropic support, with clear test addition.796a646cfix(non-interactive-env): use detectShellType instead of hardcoded win32 check ([Bug]: Force PowerShell env behavior in Windows code-yeongyu/oh-my-openagent#3310, Bug: PowerShell env var syntax ($env:VAR) used in bash shell on Windows code-yeongyu/oh-my-openagent#3338) -- The commit replaces a hardcoded win32 check with detectShellType(), improving compatibility with various shells on Windows (Git Bash, WSL) by dynamically detecting the shell type.64e55936fix(multimodal-looker): instruct explicit Read tool usage for PDFs (Multimodal-Looker Cannot Read PDF Files code-yeongyu/oh-my-openagent#2998) -- Explicit instruction added to ensure PDFs are loaded via the Read tool before extraction, addressing a concrete behavior gap.d8b9bf1afix(doctor): check comment-checker lazy-download cache path ([Question]: comment-checker not found code-yeongyu/oh-my-openagent#2911, [Bug] comment-checker hook does not trigger code-yeongyu/oh-my-openagent#3315) -- Fixes a concrete behavior issue by aligning the doctor's resolution order with runtime, ensuring cached binaries are recognized as installed.3bfa3bd6fix(background-agent): pass query directory to session.get in 4 call-sites ([Bug]: resolveSubagentSpawnContext fails to find project-scoped sessions — client.session.get() missing directory parameter code-yeongyu/oh-my-openagent#2937) -- Fixes a concrete behavior issue by passing the query directory to session.get in multiple call-sites, resolving 404 errors under newer OpenCode SDK versions with clear test coverage.eb495793fix(event): prefer real session.idle over recent synthetic dedup ([Bug]: The model has responded normally, but there is a red alert in the input box: [TODO-DIAG] BLOCKED: no todossession.idle event code-yeongyu/oh-my-openagent#2667) -- The commit fixes a concrete behavior issue by ensuring real session.idle events are not dropped when a synthetic idle has fired within the dedup window, with clear test additions.aed8dbfafix(delegate-task): fetch session result before honoring abort signal ([Bug]: @explore subagent completes but parent never returns result; 3.12.3 aborts with "The operation was aborted." code-yeongyu/oh-my-openagent#2702) -- Fixes a concrete behavior issue where the sync poller would return 'Task aborted' immediately upon an abort signal, even if a terminal assistant message was already available, by attempting a final fetch before emitting the abort message.c5c5bc36fix(delegate-task): consume fallback chain on sendSyncPrompt failure (delegate-task: No automatic retry with fallback models when subagent fails due to model error code-yeongyu/oh-my-openagent#2691) -- The commit fixes a concrete behavior issue by implementing a retry mechanism with a fallback chain for sendSyncPrompt failures, including meaningful test additions.3d2eb6e4fix(config): preserve explicit git_master overrides during merge -- Adds meaningful test coverage for preserving explicit git_master settings during merge, with clear behavior delta and concrete test cases.dac0c99efeat: add Vercel AI Gateway as a provider -- Adds Vercel AI Gateway as a recognized provider with clear integration across multiple files and test updates, providing real user value.aac17135test: add vercel provider transform tests -- Adds multiple meaningful tests for Vercel provider transformations with clear expected outputs, covering various model types and edge cases.6f72ec75test: add integration tests for Vercel AI Gateway model resolution -- Adds integration tests for Vercel AI Gateway model resolution with clear test cases and expected snapshots, providing meaningful test coverage.542dc890fix(provider): use gateway-specific model IDs for vercel transform -- Fixes behavior for Vercel AI Gateway by using correct model IDs without '-preview' suffix, with clear test case updates and logic adjustments in transform functions.34e334eafeat(provider): add vercel to all gateway-supported fallback entries -- Adds vercel provider fallback entries with clear integration and test updates, providing real user value.effae16crefactor(provider): replace per-model string replacements with generic regex -- Replaced multiple per-model string replacements with a generic regex, simplifying the code and making it more maintainable for future models.680d05a2Prometheus: keep planner primary-only -- Changes Prometheus agent mode from 'all' to 'primary' across multiple tests and config builders, aligning with the stated goal of keeping the planner primary-only, with clear test updates and direct functional impact.e3b5c2befix: address review feedback for Vercel AI Gateway PR -- Adds Vercel AI Gateway integration with CLI option, priority adjustment, and test updates, providing clear user value and functionality.01a1b141fix(delegate-task): reject primary agents in task subagent resolution -- The commit fixes a concrete behavior issue by rejecting primary agents in task subagent resolution, introducing new functions with clear utility and direct functional impact.3673b962docs(readme-ru): add telemetry and legal section translation -- Added meaningful Russian translations for telemetry and legal sections in README.ru.md, including privacy policy and terms of service links with clear user value.23d0a0dcdocs(readme-zh-cn): add telemetry and legal section translation -- Added meaningful Chinese translations for telemetry and legal sections in README.zh-cn.md, including privacy policy and terms of service links with clear user value.0ece66e3docs(readme-ja): add telemetry and legal section translation -- Added meaningful Japanese translations for telemetry and legal sections in README.ja.md, including privacy policy and terms of service links with clear user value.12ed9501docs(readme-ko): add telemetry and legal section translation -- Added meaningful Korean translations for telemetry and legal sections in README.ko.md, including privacy policy and terms of service links with clear user value.c91db2ebdocs(cli): add --vercel-ai-gateway flag documentation -- Added documentation for a new CLI flag with clear description and default value, providing direct user information.e223ab20docs(configuration): add sisyphus-junior to agent list -- Added a new agent 'sisyphus-junior' to the list of available agents in the configuration documentation, providing clear user value by expanding the set of configurable options.9342e55bdocs(installation): replace npx with bunx and update provider flags -- Updated installation docs to reflect 'bunx' usage and added relevant provider flags with clear user value.05e54444docs(overview): update model-role claims and complete category list -- The commit updates the documentation to include a complete and detailed category list for model roles, enhancing user understanding and configuration clarity.7c94ee45docs(agent-model-matching): sync fallback chains with current source -- Updated fallback chains in documentation to match current source, adding 'vercel' provider paths for clarity and consistency across agent and category configurations.b57a7975docs(ollama): verify troubleshooting steps and issue links -- Updated troubleshooting steps with verified issue links and clear documentation improvements for user value.bbbd1bd0docs(orchestration): update diagram model names and category tables -- The commit updates documentation with new, specific, and informative content (e.g., model fallback chains, category tables) that enhances user understanding and configuration clarity.4ce1dde3CLI: isolate provider model id transforms from shared mocks -- Added meaningful tests for model ID transformations across multiple providers with clear expected outputs.1b36194bfix(cli): treat missing session status as idle in run completion -- Added a test case to handle missing session status as idle, ensuring the CLI doesn't hang indefinitely, and updated the logic in pollForCompletion to return 'idle' when the session ID is missing from the statuses.b5250555fix(telemetry): guard PostHog init failures -- Adds meaningful test coverage for PostHog init failure scenarios, ensuring no-op client behavior without throwing errors.0dab3116fix: resolve 3 bugs ([Bug]: non-interactive-env hook generates PowerShell $env: syntax on Git Bash (Windows) code-yeongyu/oh-my-openagent#3366, Agent name case change breaks Todo Continuation for historical sessions code-yeongyu/oh-my-openagent#3272, [Bug]: Using oh-my-opencode via bunx causes a new oh-my-openagent.json.bak.* backup file to be created on every OpenCode startup, even when no config changes were made. code-yeongyu/oh-my-openagent#3222) -- The commit fixes three concrete bugs with clear functional impact, including detecting Git Bash via MSYSTEM, resolving legacy agent names, and optimizing config backups.62c60ae9fix: numeric skill names, ultrawork missing run_in_background, ZWSP agent lookups -- The commit fixes concrete bugs with clear functional impact, including handling numeric skill names, adding a required parameter for ultrawork tasks, and stripping invisible characters for agent name comparisons.76c5356atest(agent-config): add regression tests for agent merge priority order -- Adds meaningful regression tests for agent merge priority order with clear test cases covering various scenarios.da91c535fix(call-omo-agent): address cubic review findings and add requirement-based tests -- The commit fixes concrete issues, adds meaningful tests, and improves functionality with clear value.1e85a88dfix(agent-restrictions): restore EXPLORATION_AGENT_DENYLIST as default fallback for unknown agents -- Restores a default fallback for unknown agents, directly impacting functionality with clear value.fd28f7e6feat(agents): add agent_definitions schema, eager path resolution, and JSON agent loader -- Adds agent_definitions schema, eager path resolution, and JSON agent loader with comprehensive test suite and clear functional impact.5755a90cfeat(agents): add agent definitions file loader and opencode.json reader -- Adds agent definitions loader with comprehensive test coverage and clear functional value, including support for multiple file formats and logical loading precedence.39bda91bfeat(agents): wire agent_definitions and opencode.json agents into precedence chain -- The commit integrates agent_definitions and opencode.json agents into the precedence chain with clear functional impact, new tests, and logical filtering logic.3d0fb22cfix(call-omo-agent): adapt dev's model override tests to use createMockCtx factory -- The commit adapts existing tests to use a new factory function (createMockCtx) instead of a deprecated constant (mockCtx), ensuring test compatibility with the current codebase.42445f51fix(agents): address cubic review findings on agent loader -- The commit addresses specific cubic review findings with concrete behavior fixes, including case-insensitive file extension handling, relative path resolution, and precedence logic adjustments, all backed by new tests.cf4b2315fix(agents): replace 'in' with Object.hasOwn() for prototype-safe property checks -- Replaces 'in' with Object.hasOwn() for prototype-safe property checks in two occurrences, addressing a specific review finding to ensure accurate own-property checks on plain objects.4c77045cfix(agents): use null-prototype accumulator to prevent proto pollution -- The commit fixes a concrete security issue by preventing proto pollution using a null-prototype accumulator, directly improving code safety with clear functional value.e5d3fe96fix(agents): address all PR feat(call-omo-agent): support custom agents via dynamic resolution code-yeongyu/oh-my-openagent#2299 code review findings -- The commit addresses multiple specific code review findings with concrete changes, including security fixes (preventing prototype pollution), functional improvements (path resolution, caching), and test additions, all with clear value.ab11f2ebfix(tests): replace mock.module with spyOn to prevent test pollution -- Replaced mock.module() with spyOn() to prevent test pollution, ensuring proper restoration in afterEach()e1b2f97bfix: isPlanFamily regression tests, code-review -> review-work in test data (bug: isPlanFamily() substring matching false-positives Atlas/Metis/Momus display names code-yeongyu/oh-my-openagent#3312, [Bug]: Skill tool schema uses non-existent "code-review" as example instead of actual "review-work" skill code-yeongyu/oh-my-openagent#3285, transformModelForProvider missing anthropic provider — dash-format model IDs rejected by OpenCode code-yeongyu/oh-my-openagent#3290) -- The commit fixes regression tests by updating 'code-review' to 'review-work' in test data and adds specific tests to ensure isPlanFamily correctly handles false positives and edge cases.47aa3025fix(cli): use getAgentRuntimeName for agent resolution in run command -- The commit fixes the agent resolution in the run command by using getAgentRuntimeName, which is a concrete behavior fix with clear necessity.0764526afix(posthog): disable exception autocapture to stay within free tier -- Disables exception autocapture to stay within PostHog's free tier limit, preserving manual capture for critical errors.ccb2715afix: hide native plan agent when replace_plan is true ([Bug]: Native plan agent not hidden when replace_plan is true - Sisyphus routes to it instead of Prometheus code-yeongyu/oh-my-openagent#3443) -- The commit fixes a concrete behavior issue by hiding the native plan agent when replace_plan is true, preventing incorrect routing to the native plan agent instead of Prometheus, with a clear functional delta.a1842f2dfeat(tool-metadata): add shared metadata contract and bridge -- Adds a shared metadata contract and bridge with meaningful tests covering various scenarios, including round-trip persistence, case insensitivity, and edge cases like missing or blank call IDs.80d3339cfeat(background-agent): add wait-for-task-session helper -- Adds awaitForTaskSessionIDhelper with comprehensive tests covering immediate resolution, polling, abort signals, timeouts, and terminal task statuses, providing clear functional value for task session management.4da30057fix(plugin): harden metadata recovery and extraction -- The commit introduces meaningful functionality to extract session IDs from metadata, including legacy aliases, with clear tests and functional value.c6a407c4refactor(tools): migrate producers to shared metadata bridge -- The commit refactors multiple files to migrate producers to a shared metadata bridge, replacingstoreToolMetadatawithpublishToolMetadataand removing redundantresolveCallIDlogic, indicating a clear structural improvement with broad applicability.7bc170fbfix: installer writes hyphenated anthropic IDs, variant=max Anthropic OAuth compat (variant: max sends output_config which breaks Claude Pro/Max OAuth authentication code-yeongyu/oh-my-openagent#3429, [Bug]: installer generates invalid Anthropic model IDs with dotted version suffixes code-yeongyu/oh-my-openagent#3459) -- The commit fixes a concrete behavior issue by replacing dots with hyphens in Anthropic model IDs to ensure compatibility with Anthropic OAuth, as evidenced by the systematic replacement of '4.6' with '4-6' across all model configurations.Branch:
sync/upstream/v3.17.4-good(local only; not on origin)🟡 Review batch
Ambiguous. Human should review before merging.
cd8352c1refactor(claude-code-mcp-loader): simplify env expansion null guard -- Classification failed: No object generated: response did not match schema.569addd3docs(provider): add comments to vercel transform logic -- Classification failed: No object generated: response did not match schema.Branch:
sync/upstream/v3.17.4-needs-review(on origin)🔴 Slop batch
Classified as AI slop or pointless churn. Do not merge as-is; rewrite intent if salvageable.
1e2e0019refactor(create-tools): keep result type internal -- The commit changes the visibility ofCreateToolsResultfromexportto internal without any clear functional or architectural benefit, indicating minor visibility churn.9d89bbb0refactor(commands): keep builtin command options internal -- Changes the visibility of LoadBuiltinCommandsOptions from export to internal without clear functional or architectural benefit0c5cd3a1chore(agents): update all AGENTS.md generation dates to 2026-04-11 -- The commit updates generation dates in multiple AGENTS.md files without adding new information or functionality, indicating date metadata churn.e654cfb4@ahuangsnail has signed the CLA in fix: treat zero limit.output as unknown to enable fallback to bundled… code-yeongyu/oh-my-openagent#3316 -- The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value.6713b30crefactor(hooks): drop no-op directory injector callbacks -- Removed no-op callbacks but introduced overly complex hook interfaces with unclear value6a0d515d@divlook has signed the CLA in fix: align GitHub Copilot model variants with catalog code-yeongyu/oh-my-openagent#3353 -- The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value.5b33cd22@lrq3000 has signed the CLA in docs(install): Update installation commands for humans in documentation to oh-my-openagent code-yeongyu/oh-my-openagent#3358 -- The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value.c6deb6d6@Zireael has signed the CLA in fix: detect Git Bash/WSL/MSYS2 shell on Windows (non-interactive-env hook) code-yeongyu/oh-my-openagent#3370 -- The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value.42df1c3c@FuDesign2008 has signed the CLA in fix(agent-key): restore getAgentDisplayName for HTTP-header-safe agent keys code-yeongyu/oh-my-openagent#3371 -- Commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value.84f980c9@matchai has signed the CLA in feat(provider): add Vercel AI Gateway support code-yeongyu/oh-my-openagent#3376 -- The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value.678011c7docs(contributing): update tool/hook counts and version references -- The commit updates counts and version references in the contributing docs without adding new substantive information.cfd152addocs(features): update agent count and verify category list -- Added two new features ('review-work' and 'ai-slop-remover') with overly broad, vague descriptions lacking concrete behavior deltas or clear user value metrics.0cb0a6c0docs(contributing): update tool/hook counts and version references -- Updated counts and version references in docs without adding new substantive information.3419bbc4@mauriciozaffari has signed the CLA in fix: recognize 'hit your limit' as retryable error pattern code-yeongyu/oh-my-openagent#3398 -- CLA signature commit with no code/product value1bb59c3erelease: v3.17.2 -- This commit is a release version bump with no code changes, only updating package versions to 3.17.2 across multiple platform-specific binaries.571cf1f1@kywoo26 has signed the CLA in fix(background-agent): skip unavailable fallback models code-yeongyu/oh-my-openagent#3402 -- CLA signature commit with no code/product valuef98f7ec7@garnetlyx has signed the CLA in feat: Task ID semantics and lifecycle integrity (#3404, #3405) code-yeongyu/oh-my-openagent#3409 -- CLA signature commit with no code/product valued97c067d@lightrabbit has signed the CLA in fix(skill-mcp): pass workspace directory as cwd to stdio MCP processes code-yeongyu/oh-my-openagent#3415 -- CLA signature commit with no code/product valueeaf5ff21@fr1sk has signed the CLA in fix(agent-display): remove ZWSP sort prefixes that truncate agent names in terminals code-yeongyu/oh-my-openagent#3419 -- CLA signature commit with no code/product valuef8f7ac10@grandmaster451 has signed the CLA in fix(auto-update): use semantic version comparison instead of string e… code-yeongyu/oh-my-openagent#3420 -- CLA signature commit with no code/product value54a94d41@kithawk has signed the CLA in Fix primary agent model precedence code-yeongyu/oh-my-openagent#3428 -- CLA signature commit with no code/product value81a03fa9@orbisai0security has signed the CLA in fix: upgrade express-rate-limit to 8.2.2, 8.1.1, 8.0.2 (CVE-2026-30827) code-yeongyu/oh-my-openagent#3440 -- The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value.9aeb87c2chore: bump version to 3.17.3 -- The commit only updates the version number in package.json with no functional changes.80e73f57@CHLK has signed the CLA in fix(cli-run): prevent premature exit when background tasks are active code-yeongyu/oh-my-openagent#3455 -- The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value.fe091ef2chore: bump version to 3.17.4 -- The commit only updates the version number in package.json with no functional changes.c9350c67release: v3.17.4 -- The commit only updates version numbers in multiple package.json files with no functional changes, constituting administrative noise.Branch:
sync/upstream/v3.17.4-slop(on origin)Slop verifier second-pass notes
1e2e001CONFIRMED_SLOP (behavior delta: none): The commit changes the visibility ofCreateToolsResultfromexportto internal without any clear functional or architectural benefit, indicating minor visibility churn. There is no concrete behavior delta, and the abstraction's change in visibility does not introduce new functionality or fix an issue. The commit message accurately reflects the change but does not justify its value. A senior engineer would likely question the necessity of this change without further context on planned usage or dependencies that require this internalization.9d89bbbCONFIRMED_SLOP (behavior delta: none): The commit changes the visibility ofLoadBuiltinCommandsOptionsfromexporttointernalwithout clear functional or architectural benefit. This is classified as minor_visibility_churn, and without a concrete benefit, it remains SLOP. No behavior delta is introduced as the interface's usage scope is merely reduced, not altered in functionality.0c5cd3aCONFIRMED_SLOP (behavior delta: none): The commit updates generation dates in multiple AGENTS.md files without adding new functionality or information, indicating date metadata churn. This aligns with the 'date_count_metadata_churn' slop signal. No runtime code paths change, and the commit message accurately reflects the change but lacks measurable value. A senior engineer would likely not defend this as valuable in a code review focused on merge value for this fork.e654cfbCONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value. The first-pass reason correctly identifies it as cla_admin, a category excluded from being demoted to GOOD by the verification bias. There is no concrete behavior delta, and the change does not introduce any new functionality or fixes.6713b30CONFIRMED_SLOP (behavior delta: minor): The commit removes no-op callbacks but introduces overly complex hook interfaces without clear value. The changes do not alter runtime behavior significantly (behavior_delta: minor) and the complexity added does not have an obvious payoff or multiple callers. The commit message accurately reflects the removal of no-op code but does not justify the new complexity. A senior engineer would likely question the necessity of the introduced interfaces without a clear use case or future plan. The first-pass verdict correctly identified this as SLOP due to the workaround-heavy approach and unclear benefits.6a0d515CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value. The first-pass verdict correctly identifies it as cla_admin, a category excluded from being demoted to GOOD by the verification bias. There is no concrete behavior delta, and the change does not contribute to the functionality or quality of the codebase.5b33cd2CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value. This matches the 'cla_admin' category, which is explicitly non-GOOD by the verification bias rules.c6deb6dCONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value. This aligns with the 'cla_admin' category, which is explicitly non-GOOD by the verification bias rules. There is no behavior delta, no new functionality, and the change does not impact any runtime code path or user-visible output.42df1c3CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value. The first-pass reason correctly identifies it as cla_admin, a category excluded from being demoted to GOOD by the verification bias. No behavior delta exists as this change does not affect runtime code or user-visible output.84f980cCONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value. The first-pass reason correctly identifies this as cla_admin, a category excluded from being demoted to GOOD by the verification bias. There is no concrete behavior delta, and the change does not introduce any new functionality or improvements.678011cCONFIRMED_SLOP (behavior delta: none): The commit updates counts and version references in the contributing docs without adding new substantive information, aligning with documented slop patterns for metadata churn. No runtime behavior changes, and the updates are purely bookkeeping. A senior engineer would likely question the value given the lack of new content or functionality.cfd152aCONFIRMED_SLOP (behavior delta: none): The commit adds two new features with vague descriptions lacking concrete behavior deltas or clear user value metrics. The commit message's 'verify category list' is unmeasurable, and the changes expand existing docs without new facts. No runtime code changes occur, and the benefits are not quantifiable or directly observable.0cb0a6cCONFIRMED_SLOP (behavior delta: none): The commit updates counts and version references in documentation without adding new substantive information. The changes are purely numerical and do not introduce new facts, functionality, or value beyond keeping existing content up-to-date, which aligns with the first-pass reason for SLOP. No behavior delta exists as it's a doc change.3419bbcCONFIRMED_SLOP (behavior delta: none): The commit only updates a CLA signature in cla.json, which is administrative noise without code or product value. The first-pass reason correctly identifies it as cla_admin, a category excluded from being demoted to GOOD by the verification bias. No behavior delta exists as it doesn't alter runtime code or user-visible output.1bb59c3CONFIRMED_SLOP (behavior delta: none): The commit is a release version bump from 3.17.0 to 3.17.2 with no code changes, only updating version numbers in multiple package.json files. This matches the 'release_version_bump' slop pattern. There is no concrete behavior delta, as no runtime code paths or user-visible outputs are altered. The commit's sole purpose is administrative version updating, which is classified as SLOP by the given rubric.571cf1fCONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json without any code or product changes, aligning with documented slop patterns for administrative noise. No behavior delta exists as it doesn't affect runtime code or user-visible output. The commit message accurately reflects the change but lacks value beyond administrative tracking.f98f7ecCONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json, which is administrative noise with no direct code or product value. The first-pass verdict correctly identifies it as SLOP based on the documented pattern for CLA signatures. No behavior delta exists as this change does not affect runtime code or user-visible output.d97c067CONFIRMED_SLOP (behavior delta: none): The commit only updates a CLA signature in cla.json without any code or product changes. This matches the 'cla_admin' category, which is not considered GOOD by default. There's no behavior delta, and the change is administrative noise.eaf5ff2CONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json, which is administrative noise with no direct code or product value. The first-pass verdict correctly identifies it as cla_admin, a category excluded from being demoted to GOOD by the verification bias rules. No behavior delta exists as this change does not affect runtime code or user-visible output.f8f7ac1CONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json, which is administrative noise with no direct code or product value. The first-pass verdict correctly identifies it as cla_admin, a category excluded from being demoted to GOOD by the verification bias rules.54a94d4CONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json with no code or product changes, aligning with documented slop patterns for administrative noise. No behavior delta exists as it's purely metadata. The first-pass verdict correctly identifies it as SLOP due to lacking merge value for the fork.81a03faCONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json, which is purely administrative with no impact on code functionality or product value. This aligns with the 'cla_admin' slop pattern, and there's no concrete behavior delta or value addition. The first-pass verdict correctly identifies it as SLOP.9aeb87cCONFIRMED_SLOP (behavior delta: none): The commit only updates the version number in package.json with no functional changes, aligning with the 'release_version_bump' slop pattern. No behavior delta exists as it's purely administrative. A senior engineer would likely not defend this as valuable in a code review focused on functional merge value. The commit message accurately reflects the change but lacks measurable value beyond administrative tracking.80e73f5CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value. The first-pass reason correctly identifies it as cla_admin, a category excluded from being demoted to GOOD by the verification bias. No behavior delta exists as this change does not affect runtime code or user-visible output.fe091efCONFIRMED_SLOP (behavior delta: none): The commit only updates the version number in package.json with no functional changes, aligning with the 'release_version_bump' slop pattern. No behavior delta exists as it's purely administrative. A senior engineer would likely not defend this as valuable in a code review focused on functional merge value.c9350c6CONFIRMED_SLOP (behavior delta: none): The commit only updates version numbers in multiple package.json files with no functional changes, constituting administrative noise (release_version_bump). There is no concrete behavior delta, and the change is purely administrative.Fork-sync impact assessment
Breaking changes
(none detected)
Dependency changes
(none)
Architecture drift
(none)
Hidden concerns
Action items before merging
Generated by
upstream-analyzerworkflow. This issue is advisory: no merges or publishes happen automatically. Phase 1: draft PRs only. The prompt lives in.github/prompts/and is editable without touching workflow code.Classify chain:
openrouter:qwen/qwen3-coder:free → nvidia:nvidia/llama-3.3-nemotron-super-49b-v1 → github:openai/gpt-4.1-miniSlop-verify chain:
openrouter:nvidia/nemotron-3-super-120b-a12b:free → nvidia:nvidia/llama-3.3-nemotron-super-49b-v1 → github:openai/gpt-4.1Synthesis chain:
openrouter:nvidia/nemotron-3-super-120b-a12b:free → nvidia:nvidia/llama-3.3-nemotron-super-49b-v1 → github:openai/gpt-4.1