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 tangible benefits, including bug fixes, feature enhancements, and documentation improvements. However, the SLOP ratio (22.4%) indicates a substantial amount of non-value or low-value changes, including administrative noise (CLA signatures) and minor refactor churn. While the GOOD commits outweigh the SLOP, the presence of administrative and minor changes suggests a targeted approach to merging. No critical breaking changes or dependency conflicts were identified that would block a selective merge.
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 user and project agents to subagent resolution with clear logic and 4 new meaningful tests covering different 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 when names collide, with a new test added to validate this logic.
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 when names collide, with a new test added to validate this logic.
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 stated benefit of being more idiomatic for bun:test, as suggested in a PR review.
e9b6bba1refactor(agent-display): centralize name normalization resolution -- Adds meaningful test coverage for legacy name conversion and strips UI prefixes, with clear functional benefit in agent name normalization.
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 test coverage for shared gateway URL validation scenarios, ensuring the function behaves as expected with various URL types.
a2ae3de3refactor(openclaw): extract gateway url validator -- Extracted a reusable function validateGatewayUrl into a separate file, reducing code duplication and improving maintainability with clear functional benefit.
69f1c9aefix(doctor): add timeouts to subprocess spawns to prevent exit code 137 -- Adds concrete timeouts to prevent subprocess hangs, directly addressing a specific error (exit code 137) with measurable improvement.
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 concrete timeouts to prevent subprocess hangs, directly addressing a specific error (exit code 137) with measurable improvement.
565d3ffafix(ci): build darwin binaries on macOS runners to restore code signing -- The commit fixes a concrete CI issue by routing darwin platforms to macOS runners for native compilation and adds an explicit ad-hoc codesign step to restore code signing, directly addressing a functional problem with macOS Gatekeeper.
71b5ab67fix(ci): replace breaking codesign step with signature verification -- Replaces a breaking CI step with a necessary signature verification, directly addressing a functional issue with macOS builds.
16e7fa5bfix(ci): ad-hoc sign darwin binaries with plain codesign -- The commit fixes a concrete CI issue by adding an explicit ad-hoc codesign step for darwin binaries on GitHub macOS runners, directly addressing a functional problem with code signing.
470ed8b1fix(ci): remove existing signature before ad-hoc signing darwin binary -- The commit fixes a concrete CI issue by removing an existing malformed signature before applying a fresh ad-hoc signature to darwin binaries, directly addressing a functional problem with code signing.
2d65896brefactor(shared): simplify normalize SDK null guards -- Simplifies null checks in normalizeSDKResponse from two separate checks to a single '== null' check, improving readability without changing behavior.
79a475d6refactor(background-agent): simplify loop detector null guard -- Simplifies null checks in sortObject from two separate checks to a single '== null' check, improving readability without changing behavior.
50df6f0dtest(claude-code-plugin-loader): cover plugin path nullish resolution -- Adds meaningful test coverage for plugin path resolution, including nullish input handling.
668bc8e8refactor(config-manager): simplify config parsing guards -- Simplifies redundant conditional logic in config parsing and adds meaningful test coverage for valid and invalid config scenarios.
a0d5131erefactor(plugin): remove dead chat params code -- Removes unused 'variant' parameter and dead code related to it in chat-params.ts, simplifying the buildChatParamsInput function.
21cad26cdocs(config): remove redundant schema default comments -- Removed redundant default value comments in schema definitions, improving clarity without changing behavior.
4de02094test(background-agent): lock nullish loop detector behavior -- Adds meaningful test coverage for nullish input handling in loop detector behavior, ensuring consistent output for null and undefined inputs.
141798efrefactor(background-agent): standardize loop detector null guards -- Simplifies null checks in two functions from separate undefined/null checks to a single == null check, improving readability without changing behavior.
3d2eb6e4fix(config): preserve explicit git_master overrides during merge -- Adds tests and logic to preserve explicit git_master overrides, fixing a concrete behavior issue with config merging.
dac0c99efeat: add Vercel AI Gateway as a provider -- Adds Vercel AI Gateway as a recognized provider with clear integration across multiple files, enabling users to use it as a universal fallback for various models.
aac17135test: add vercel provider transform tests -- Adds meaningful test coverage for Vercel provider transformations with multiple specific scenarios.
6f72ec75test: add integration tests for Vercel AI Gateway model resolution -- Adds integration tests for Vercel AI Gateway model resolution with specific scenarios and snapshot updates, increasing test coverage for a real feature.
542dc890fix(provider): use gateway-specific model IDs for vercel transform -- Fixes a concrete behavior issue by using the correct gateway-specific model ID for Vercel transform, with clear functional delta in tests and code.
34e334eafeat(provider): add vercel to all gateway-supported fallback entries -- Adds vercel provider fallback entries with clear integration and test updates, enabling real user value through gateway-supported models.
effae16crefactor(provider): replace per-model string replacements with generic regex -- Replaces multiple per-model string replacements with a generic regex, simplifying code and making it more scalable for new models without changes.
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 functional delta in tests and code.
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.
01a1b141fix(delegate-task): reject primary agents in task subagent resolution -- The commit introduces a concrete behavior fix by rejecting primary agents in task subagent resolution, with clear functional delta in code and tests.
3673b962docs(readme-ru): add telemetry and legal section translation -- Adds meaningful Russian translation for telemetry and legal sections in README.ru.md, including functional details and privacy policy links.
23d0a0dcdocs(readme-zh-cn): add telemetry and legal section translation -- Adds meaningful Chinese translation for telemetry and legal sections in README.zh-cn.md, including functional details and privacy policy links.
0ece66e3docs(readme-ja): add telemetry and legal section translation -- Adds meaningful Japanese translation for telemetry and legal sections in README.ja.md, including functional details and privacy policy links.
12ed9501docs(readme-ko): add telemetry and legal section translation -- Adds meaningful Korean translation for telemetry and legal sections in README.ko.md, including functional details and privacy policy links.
c91db2ebdocs(cli): add --vercel-ai-gateway flag documentation -- Adds documentation for a new CLI flag --vercel-ai-gateway with clear description and default value, providing user value.
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 -- The commit updates installation documentation to replace 'npx' with 'bunx' and adds new provider flags with clear user value, improving the installation process.
05e54444docs(overview): update model-role claims and complete category list -- The commit updates documentation with a clear expansion of category lists and model mappings, adding concrete new information for users.
7c94ee45docs(agent-model-matching): sync fallback chains with current source -- The commit updates documentation to reflect accurate fallback chains by adding 'vercel' to model IDs, providing clear user value through precise configuration details.
b57a7975docs(ollama): verify troubleshooting steps and issue links -- Updated troubleshooting steps with specific issue links and clear workarounds, adding concrete user value.
bbbd1bd0docs(orchestration): update diagram model names and category tables -- The commit updates documentation with clear, concrete changes to model names, category tables, and fallback chains, providing substantive user value through improved accuracy and 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 -- The commit fixes a concrete behavior issue by treating missing session status as idle, preventing indefinite hanging, with a clear test case added.
b5250555fix(telemetry): guard PostHog init failures -- The commit adds meaningful test coverage for a real scenario where PostHog initialization failures are guarded, ensuring no-op behavior without errors.
62c60ae9fix: numeric skill names, ultrawork missing run_in_background, ZWSP agent lookups -- The commit fixes concrete bugs with clear functional deltas, including coercing numeric skill names to strings, adding a required parameter to task examples, and stripping ZWSP from agent names for accurate lookups.
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 and adds meaningful tests with clear functional deltas.
1e85a88dfix(agent-restrictions): restore EXPLORATION_AGENT_DENYLIST as default fallback for unknown agents -- Restores a default fallback for unknown agents, fixing a concrete behavior delta in agent restrictions.
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 clear functional value and test coverage.
5755a90cfeat(agents): add agent definitions file loader and opencode.json reader -- Adds agent definitions loader with multiple file format support, comprehensive tests, and clear functional value for loading and parsing agent configurations.
39bda91bfeat(agents): wire agent_definitions and opencode.json agents into precedence chain -- Adds integration of agent_definitions and opencode.json agents into precedence chain with clear functional value, new tests, and filtering logic.
3d0fb22cfix(call-omo-agent): adapt dev's model override tests to use createMockCtx factory -- The commit fixes a concrete compatibility issue by replacing outdated mockCtx references with the new createMockCtx factory in four test cases, ensuring tests work with the updated codebase.
42445f51fix(agents): address cubic review findings on agent loader -- Fixes concrete issues with agent loading, including case-insensitive file extension handling, relative path resolution, and precedence logic, with clear functional deltas and new test coverage.
cf4b2315fix(agents): replace 'in' with Object.hasOwn() for prototype-safe property checks -- Replaced 'in' with Object.hasOwn() for prototype-safe property checks in two occurrences, addressing a concrete correctness issue with inherited properties.
4c77045cfix(agents): use null-prototype accumulator to prevent proto pollution -- The commit fixes a concrete security issue by using a null-prototype accumulator to prevent proto pollution, addressing a specific vulnerability with clear functional delta.
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 concrete issues and improvements, including security fixes (prototype pollution prevention), functional fixes (empty path validation), and meaningful refactorings (extracting shared parseToolsConfig) with clear benefits and new test coverage.
ab11f2ebfix(tests): replace mock.module with spyOn to prevent test pollution -- Replaced mock.module() with spyOn() to prevent test state pollution, ensuring downstream tests run cleanly.
47aa3025fix(cli): use getAgentRuntimeName for agent resolution in run command -- The commit fixes a concrete behavior issue by replacing getAgentDisplayName with getAgentRuntimeName for agent resolution in the run command, with clear functional delta and updated tests.
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 preservation, casing insensitivity, and edge cases like blank or missing call IDs.
80d3339cfeat(background-agent): add wait-for-task-session helper -- Adds a waitForTaskSessionID helper with meaningful tests covering immediate resolution, polling, abort signals, and timeouts, providing clear 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 test coverage and functional delta in multiple files.
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 call ID resolution 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, indicating minor visibility churn.
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 functionality or content, indicating date metadata churn.
678011c7docs(contributing): update tool/hook counts and version references -- The commit updates counts and version references in the documentation without adding new information or functionality, characteristic of date/count metadata churn.
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, characteristic of AI-generated fluff.
0cb0a6c0docs(contributing): update tool/hook counts and version references -- The commit updates counts and version references in the documentation without adding new information or functionality.
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, indicating 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. This is classified as minor visibility churn, which is a documented slop pattern. There's no concrete behavior delta, and the commit message lacks justification for the change. A senior engineer would likely question the value of this change without further context.
9d89bbb CONFIRMED_SLOP (behavior delta: none): The commit changes the visibility of LoadBuiltinCommandsOptions from export to internal without clear functional or architectural benefit, indicating minor visibility churn. There's no concrete behavior delta, as the interface's usage scope is merely reduced without altering its structure or any dependent code's behavior. The commit message lacks justification for this change, and no obvious growth path or multiple callers are evident for this interface in its new internal scope.
0c5cd3a CONFIRMED_SLOP (behavior delta: none): The commit updates generation dates in multiple AGENTS.md files without adding new functionality or content, indicating date metadata churn. This aligns with the first-pass reason and documented slop patterns for date updates. 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 functional or behavioral changes.
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 verdict 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.
6713b30 CONFIRMED_SLOP (behavior delta: none): The commit removes unused 'tool.execute.before' callbacks in two hook files, which the first-pass correctly identified as refactor_churn with no functional impact. The changes are purely internal with no user-visible output or behavior delta. The commit message accurately reflects the removal but does not justify value beyond churn. A senior engineer would likely question the necessity of this change without broader context or future plans, aligning with the initial SLOP verdict.
6a0d515 CONFIRMED_SLOP (behavior delta: none): The commit adds only a CLA signature entry to cla.json, which is administrative noise without code or product value, aligning with the 'cla_admin' slop signal. No behavior delta exists as it doesn't alter runtime code or user-visible output. The commit message accurately reflects the change but lacks technical merit. A senior engineer would likely not defend this as valuable in a code review focused on merge value.
5b33cd2 CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature entry to cla.json, which is administrative noise with no code or product value. This aligns with the 'cla_admin' slop signal. There is no concrete behavior delta, and the change does not improve functionality or user experience.
c6deb6d CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature entry to cla.json, which is administrative noise with no code or product value. This aligns with the 'cla_admin' category, which is explicitly non-mergable as per the verification bias. No behavior delta exists, and the change does not impact runtime code or user-visible output.
42df1c3 CONFIRMED_SLOP (behavior delta: none): The commit adds only a CLA signature entry to cla.json, which is administrative noise without code or product value, aligning with the 'cla_admin' slop signal. No behavior delta exists as it doesn't alter runtime code or user-visible output. The commit message accurately reflects the change but doesn't provide measurable value. A senior engineer would likely not defend this as valuable in a code review focused on merge value for the fork.
84f980c CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature entry to cla.json, which is administrative noise with no code or product value. This aligns with the 'cla_admin' category, which is explicitly non-mergable as per the verification bias. No behavior delta exists as it doesn't affect runtime code or user-visible output.
678011c CONFIRMED_SLOP (behavior delta: none): The commit updates counts and version references in documentation without adding new functionality or substantive information. This aligns with date_count_metadata_churn, a documented slop pattern. No runtime behavior changes, and the commit message accurately reflects the metadata updates without claiming broader improvements.
cfd152a CONFIRMED_SLOP (behavior delta: none): The commit adds two new features with vague descriptions lacking concrete behavior deltas or clear user value, characteristic of AI-generated fluff. The features 'review-work' and 'ai-slop-remover' introduce new abstractions with exactly one caller and no clear growth path, while the docs expand existing content without adding new facts. No concrete behavior delta is introduced, and the commit message's claims are not supported by measurable changes.
0cb0a6c CONFIRMED_SLOP (behavior delta: none): The commit updates counts and version references in the documentation without adding new information or functionality, aligning with 'date_count_metadata_churn'. No runtime behavior changes, and the update is purely informational with no clear growth path or measurable benefit beyond accuracy. A senior engineer would likely question the necessity of this change as a standalone commit due to its minimal impact.
3419bbc 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 rules. There is no behavior delta, no new functionality, and the change does not impact runtime code paths or user-visible output.
1bb59c3 CONFIRMED_SLOP (behavior delta: none): This commit only updates version numbers in multiple package.json files without any code changes, fitting the 'release_version_bump' category which is excluded from being demoted to GOOD by the verification bias. There is no concrete behavior delta, and the commit's sole purpose is administrative metadata update.
571cf1f CONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json, which is purely administrative with no impact on code or product functionality. This aligns with the 'cla_admin' slop signal and does not provide measurable value to the repository's functionality or user experience.
f98f7ec CONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json, which is purely administrative and does not introduce any code or product value. This aligns with the 'cla_admin' slop signal, indicating no concrete behavior delta or measurable improvement.
d97c067 CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is purely administrative with no impact on code or product functionality. This aligns with the 'cla_admin' slop signal and does not provide any measurable value to the repository's functionality or quality.
eaf5ff2 CONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json, which is purely administrative and does not introduce any code or product value. The first-pass verdict correctly identifies this as cla_admin, a category excluded from being demoted to GOOD by the verification bias. There is no behavior delta, and the change does not strengthen or weaken any invariants.
f8f7ac1 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-mergable as per the verification bias rules. No behavior delta exists, and the change does not introduce any new functionality or improvements.
54a94d4 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 change does not alter any runtime behavior, user-visible output, or invariants. It fits the cla_admin category, which is excluded from being demoted to GOOD by the verification bias.
81a03fa 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' slop signal and does not provide any concrete behavior delta, abstractions, or user-visible changes. As per the verification bias, cla_admin commits are not considered for demotion to GOOD.
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. The first-pass verdict correctly identifies it as SLOP following the bias against release version bumps.
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. This aligns with the 'cla_admin' category, which is explicitly non-mergable as per the verification bias. No behavior delta exists, and the change does not improve functionality or correctness.
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, indicating administrative noise. This aligns with the 'release_version_bump' category, which is not considered GOOD by default. 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 may require careful review for unintended side effects
Action items before merging
Review refactor commits for unintended side effects
Verify CLA signatures are not inadvertently merged as part of code changes
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 tangible benefits, including bug fixes, feature enhancements, and documentation improvements. However, the SLOP ratio (22.4%) indicates a substantial amount of non-value or low-value changes, including administrative noise (CLA signatures) and minor refactor churn. While the GOOD commits outweigh the SLOP, the presence of administrative and minor changes suggests a targeted approach to merging. No critical breaking changes or dependency conflicts were identified that would block a selective merge.
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 user and project agents to subagent resolution with clear logic and 4 new meaningful tests covering different 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 when names collide, with a new test added to validate this logic.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 when names collide, with a new test added to validate this logic.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 stated benefit of being more idiomatic for 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 to handle drive-letter paths and CRLF line endings, resolving a reported bug ([Bug]: Grep tool content mode always returns 'No matches found' on Windows — drive letter breaks regex parsing code-yeongyu/oh-my-openagent#2962).e9b6bba1refactor(agent-display): centralize name normalization resolution -- Adds meaningful test coverage for legacy name conversion and strips UI prefixes, with clear functional benefit in agent name normalization.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 test coverage for shared gateway URL validation scenarios, ensuring the function behaves as expected with various URL types.a2ae3de3refactor(openclaw): extract gateway url validator -- Extracted a reusable function validateGatewayUrl into a separate file, reducing code duplication and improving maintainability with clear functional benefit.69f1c9aefix(doctor): add timeouts to subprocess spawns to prevent exit code 137 -- Adds concrete timeouts to prevent subprocess hangs, directly addressing a specific error (exit code 137) with measurable improvement.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 concrete timeouts to prevent subprocess hangs, directly addressing a specific error (exit code 137) with measurable improvement.565d3ffafix(ci): build darwin binaries on macOS runners to restore code signing -- The commit fixes a concrete CI issue by routing darwin platforms to macOS runners for native compilation and adds an explicit ad-hoc codesign step to restore code signing, directly addressing a functional problem with macOS Gatekeeper.71b5ab67fix(ci): replace breaking codesign step with signature verification -- Replaces a breaking CI step with a necessary signature verification, directly addressing a functional issue with macOS builds.16e7fa5bfix(ci): ad-hoc sign darwin binaries with plain codesign -- The commit fixes a concrete CI issue by adding an explicit ad-hoc codesign step for darwin binaries on GitHub macOS runners, directly addressing a functional problem with code signing.470ed8b1fix(ci): remove existing signature before ad-hoc signing darwin binary -- The commit fixes a concrete CI issue by removing an existing malformed signature before applying a fresh ad-hoc signature to darwin binaries, directly addressing a functional problem with code signing.2d65896brefactor(shared): simplify normalize SDK null guards -- Simplifies null checks in normalizeSDKResponse from two separate checks to a single '== null' check, improving readability without changing behavior.79a475d6refactor(background-agent): simplify loop detector null guard -- Simplifies null checks insortObjectfrom two separate checks to a single'== null'check, improving readability without changing behavior.50df6f0dtest(claude-code-plugin-loader): cover plugin path nullish resolution -- Adds meaningful test coverage for plugin path resolution, including nullish input handling.668bc8e8refactor(config-manager): simplify config parsing guards -- Simplifies redundant conditional logic in config parsing and adds meaningful test coverage for valid and invalid config scenarios.a0d5131erefactor(plugin): remove dead chat params code -- Removes unused 'variant' parameter and dead code related to it in chat-params.ts, simplifying the buildChatParamsInput function.21cad26cdocs(config): remove redundant schema default comments -- Removed redundant default value comments in schema definitions, improving clarity without changing behavior.4de02094test(background-agent): lock nullish loop detector behavior -- Adds meaningful test coverage for nullish input handling in loop detector behavior, ensuring consistent output for null and undefined inputs.141798efrefactor(background-agent): standardize loop detector null guards -- Simplifies null checks in two functions from separateundefined/nullchecks to a single== nullcheck, improving readability without changing behavior.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 reported failure due to secondary archive status, 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 the scenario.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 to ensure correct migration, fixing a real 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 package references, 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) -- Fixes concrete behavior issues by replacing non-existent/plancommand references with Prometheus delegation guidance and updates a non-existent skill name to a real one (code-review->review-work)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(), adding tests for various shell environments (PowerShell, Git Bash, WSL, cmd) on Windows and Linux, ensuring correct command syntax based on actual 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, fixing 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 doctor resolution order with runtime to recognize cached binaries, with clear test addition.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) -- The commit 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 added.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) -- The commit fixes a concrete behavior issue by fetching session results before honoring the abort signal, ensuring completion results are returned if available, with clear test additions.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 fallback chains for failed sendSyncPrompt calls, including meaningful test additions.3d2eb6e4fix(config): preserve explicit git_master overrides during merge -- Adds tests and logic to preserve explicit git_master overrides, fixing a concrete behavior issue with config merging.dac0c99efeat: add Vercel AI Gateway as a provider -- Adds Vercel AI Gateway as a recognized provider with clear integration across multiple files, enabling users to use it as a universal fallback for various models.aac17135test: add vercel provider transform tests -- Adds meaningful test coverage for Vercel provider transformations with multiple specific scenarios.6f72ec75test: add integration tests for Vercel AI Gateway model resolution -- Adds integration tests for Vercel AI Gateway model resolution with specific scenarios and snapshot updates, increasing test coverage for a real feature.542dc890fix(provider): use gateway-specific model IDs for vercel transform -- Fixes a concrete behavior issue by using the correct gateway-specific model ID for Vercel transform, with clear functional delta in tests and code.34e334eafeat(provider): add vercel to all gateway-supported fallback entries -- Adds vercel provider fallback entries with clear integration and test updates, enabling real user value through gateway-supported models.effae16crefactor(provider): replace per-model string replacements with generic regex -- Replaces multiple per-model string replacements with a generic regex, simplifying code and making it more scalable for new models without changes.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 functional delta in tests and code.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.01a1b141fix(delegate-task): reject primary agents in task subagent resolution -- The commit introduces a concrete behavior fix by rejecting primary agents in task subagent resolution, with clear functional delta in code and tests.3673b962docs(readme-ru): add telemetry and legal section translation -- Adds meaningful Russian translation for telemetry and legal sections in README.ru.md, including functional details and privacy policy links.23d0a0dcdocs(readme-zh-cn): add telemetry and legal section translation -- Adds meaningful Chinese translation for telemetry and legal sections in README.zh-cn.md, including functional details and privacy policy links.0ece66e3docs(readme-ja): add telemetry and legal section translation -- Adds meaningful Japanese translation for telemetry and legal sections in README.ja.md, including functional details and privacy policy links.12ed9501docs(readme-ko): add telemetry and legal section translation -- Adds meaningful Korean translation for telemetry and legal sections in README.ko.md, including functional details and privacy policy links.c91db2ebdocs(cli): add --vercel-ai-gateway flag documentation -- Adds documentation for a new CLI flag --vercel-ai-gateway with clear description and default value, providing user value.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 -- The commit updates installation documentation to replace 'npx' with 'bunx' and adds new provider flags with clear user value, improving the installation process.05e54444docs(overview): update model-role claims and complete category list -- The commit updates documentation with a clear expansion of category lists and model mappings, adding concrete new information for users.7c94ee45docs(agent-model-matching): sync fallback chains with current source -- The commit updates documentation to reflect accurate fallback chains by adding 'vercel' to model IDs, providing clear user value through precise configuration details.b57a7975docs(ollama): verify troubleshooting steps and issue links -- Updated troubleshooting steps with specific issue links and clear workarounds, adding concrete user value.bbbd1bd0docs(orchestration): update diagram model names and category tables -- The commit updates documentation with clear, concrete changes to model names, category tables, and fallback chains, providing substantive user value through improved accuracy and 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 -- The commit fixes a concrete behavior issue by treating missing session status as idle, preventing indefinite hanging, with a clear test case added.b5250555fix(telemetry): guard PostHog init failures -- The commit adds meaningful test coverage for a real scenario where PostHog initialization failures are guarded, ensuring no-op behavior without 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 deltas, 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 deltas, including coercing numeric skill names to strings, adding a required parameter to task examples, and stripping ZWSP from agent names for accurate lookups.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 and adds meaningful tests with clear functional deltas.1e85a88dfix(agent-restrictions): restore EXPLORATION_AGENT_DENYLIST as default fallback for unknown agents -- Restores a default fallback for unknown agents, fixing a concrete behavior delta in agent restrictions.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 clear functional value and test coverage.5755a90cfeat(agents): add agent definitions file loader and opencode.json reader -- Adds agent definitions loader with multiple file format support, comprehensive tests, and clear functional value for loading and parsing agent configurations.39bda91bfeat(agents): wire agent_definitions and opencode.json agents into precedence chain -- Adds integration of agent_definitions and opencode.json agents into precedence chain with clear functional value, new tests, and filtering logic.3d0fb22cfix(call-omo-agent): adapt dev's model override tests to use createMockCtx factory -- The commit fixes a concrete compatibility issue by replacing outdated mockCtx references with the new createMockCtx factory in four test cases, ensuring tests work with the updated codebase.42445f51fix(agents): address cubic review findings on agent loader -- Fixes concrete issues with agent loading, including case-insensitive file extension handling, relative path resolution, and precedence logic, with clear functional deltas and new test coverage.cf4b2315fix(agents): replace 'in' with Object.hasOwn() for prototype-safe property checks -- Replaced 'in' with Object.hasOwn() for prototype-safe property checks in two occurrences, addressing a concrete correctness issue with inherited properties.4c77045cfix(agents): use null-prototype accumulator to prevent proto pollution -- The commit fixes a concrete security issue by using a null-prototype accumulator to prevent proto pollution, addressing a specific vulnerability with clear functional delta.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 concrete issues and improvements, including security fixes (prototype pollution prevention), functional fixes (empty path validation), and meaningful refactorings (extracting shared parseToolsConfig) with clear benefits and new test coverage.ab11f2ebfix(tests): replace mock.module with spyOn to prevent test pollution -- Replaced mock.module() with spyOn() to prevent test state pollution, ensuring downstream tests run cleanly.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 a regression in test data by updating 'code-review' to 'review-work' in test expectations and adds new tests to prevent false positives for specific plan family names.47aa3025fix(cli): use getAgentRuntimeName for agent resolution in run command -- The commit fixes a concrete behavior issue by replacinggetAgentDisplayNamewithgetAgentRuntimeNamefor agent resolution in the run command, with clear functional delta and updated tests.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, with a clear functional delta in the buildPlanDemoteConfig() function and updated tests.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 preservation, casing insensitivity, and edge cases like blank or missing call IDs.80d3339cfeat(background-agent): add wait-for-task-session helper -- Adds awaitForTaskSessionIDhelper with meaningful tests covering immediate resolution, polling, abort signals, and timeouts, providing clear 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 test coverage and functional delta in multiple files.c6a407c4refactor(tools): migrate producers to shared metadata bridge -- The commit refactors multiple files to migrate producers to a shared metadata bridge, replacingstoreToolMetadatawithpublishToolMetadataand removing redundant call ID resolution logic, 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 across all model references in the snapshot file.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 ofLoadBuiltinCommandsOptionsfromexportto internal without clear functional or architectural benefit, indicating minor visibility churn.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 functionality or content, 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 -- Removes unused 'tool.execute.before' callback with no functional impact, indicating churn.6a0d515d@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 entry 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 entry 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 entry 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 -- The commit only adds a CLA signature entry 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 entry 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 documentation without adding new information or functionality, characteristic of date/count metadata churn.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, characteristic of AI-generated fluff.0cb0a6c0docs(contributing): update tool/hook counts and version references -- The commit updates counts and version references in the documentation without adding new information or functionality.3419bbc4@mauriciozaffari has signed the CLA in fix: recognize 'hit your limit' as retryable error pattern code-yeongyu/oh-my-openagent#3398 -- The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value.1bb59c3erelease: v3.17.2 -- The commit is a release version bump with no code changes, only updating version numbers in multiple package.json files.571cf1f1@kywoo26 has signed the CLA in fix(background-agent): skip unavailable fallback models code-yeongyu/oh-my-openagent#3402 -- The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value.f98f7ec7@garnetlyx has signed the CLA in feat: Task ID semantics and lifecycle integrity (#3404, #3405) code-yeongyu/oh-my-openagent#3409 -- Commit adds a CLA signature to cla.json, which is administrative noise with no code or product value.d97c067d@lightrabbit has signed the CLA in fix(skill-mcp): pass workspace directory as cwd to stdio MCP processes code-yeongyu/oh-my-openagent#3415 -- The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value.eaf5ff21@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 -- The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value.f8f7ac10@grandmaster451 has signed the CLA in fix(auto-update): use semantic version comparison instead of string e… code-yeongyu/oh-my-openagent#3420 -- Commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value.54a94d41@kithawk has signed the CLA in Fix primary agent model precedence code-yeongyu/oh-my-openagent#3428 -- The commit only adds a CLA signature to cla.json, which is administrative noise with no code or product value.81a03fa9@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, indicating 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. This is classified as minor visibility churn, which is a documented slop pattern. There's no concrete behavior delta, and the commit message lacks justification for the change. A senior engineer would likely question the value of this change without further context.9d89bbbCONFIRMED_SLOP (behavior delta: none): The commit changes the visibility ofLoadBuiltinCommandsOptionsfromexportto internal without clear functional or architectural benefit, indicating minor visibility churn. There's no concrete behavior delta, as the interface's usage scope is merely reduced without altering its structure or any dependent code's behavior. The commit message lacks justification for this change, and no obvious growth path or multiple callers are evident for this interface in its new internal scope.0c5cd3aCONFIRMED_SLOP (behavior delta: none): The commit updates generation dates in multiple AGENTS.md files without adding new functionality or content, indicating date metadata churn. This aligns with the first-pass reason and documented slop patterns for date updates. 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 functional or behavioral changes.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 verdict 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.6713b30CONFIRMED_SLOP (behavior delta: none): The commit removes unused 'tool.execute.before' callbacks in two hook files, which the first-pass correctly identified as refactor_churn with no functional impact. The changes are purely internal with no user-visible output or behavior delta. The commit message accurately reflects the removal but does not justify value beyond churn. A senior engineer would likely question the necessity of this change without broader context or future plans, aligning with the initial SLOP verdict.6a0d515CONFIRMED_SLOP (behavior delta: none): The commit adds only a CLA signature entry to cla.json, which is administrative noise without code or product value, aligning with the 'cla_admin' slop signal. No behavior delta exists as it doesn't alter runtime code or user-visible output. The commit message accurately reflects the change but lacks technical merit. A senior engineer would likely not defend this as valuable in a code review focused on merge value.5b33cd2CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature entry to cla.json, which is administrative noise with no code or product value. This aligns with the 'cla_admin' slop signal. There is no concrete behavior delta, and the change does not improve functionality or user experience.c6deb6dCONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature entry to cla.json, which is administrative noise with no code or product value. This aligns with the 'cla_admin' category, which is explicitly non-mergable as per the verification bias. No behavior delta exists, and the change does not impact runtime code or user-visible output.42df1c3CONFIRMED_SLOP (behavior delta: none): The commit adds only a CLA signature entry to cla.json, which is administrative noise without code or product value, aligning with the 'cla_admin' slop signal. No behavior delta exists as it doesn't alter runtime code or user-visible output. The commit message accurately reflects the change but doesn't provide measurable value. A senior engineer would likely not defend this as valuable in a code review focused on merge value for the fork.84f980cCONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature entry to cla.json, which is administrative noise with no code or product value. This aligns with the 'cla_admin' category, which is explicitly non-mergable as per the verification bias. No behavior delta exists as it doesn't affect runtime code or user-visible output.678011cCONFIRMED_SLOP (behavior delta: none): The commit updates counts and version references in documentation without adding new functionality or substantive information. This aligns with date_count_metadata_churn, a documented slop pattern. No runtime behavior changes, and the commit message accurately reflects the metadata updates without claiming broader improvements.cfd152aCONFIRMED_SLOP (behavior delta: none): The commit adds two new features with vague descriptions lacking concrete behavior deltas or clear user value, characteristic of AI-generated fluff. The features 'review-work' and 'ai-slop-remover' introduce new abstractions with exactly one caller and no clear growth path, while the docs expand existing content without adding new facts. No concrete behavior delta is introduced, and the commit message's claims are not supported by measurable changes.0cb0a6cCONFIRMED_SLOP (behavior delta: none): The commit updates counts and version references in the documentation without adding new information or functionality, aligning with 'date_count_metadata_churn'. No runtime behavior changes, and the update is purely informational with no clear growth path or measurable benefit beyond accuracy. A senior engineer would likely question the necessity of this change as a standalone commit due to its minimal impact.3419bbcCONFIRMED_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 rules. There is no behavior delta, no new functionality, and the change does not impact runtime code paths or user-visible output.1bb59c3CONFIRMED_SLOP (behavior delta: none): This commit only updates version numbers in multiple package.json files without any code changes, fitting the 'release_version_bump' category which is excluded from being demoted to GOOD by the verification bias. There is no concrete behavior delta, and the commit's sole purpose is administrative metadata update.571cf1fCONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json, which is purely administrative with no impact on code or product functionality. This aligns with the 'cla_admin' slop signal and does not provide measurable value to the repository's functionality or user experience.f98f7ecCONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json, which is purely administrative and does not introduce any code or product value. This aligns with the 'cla_admin' slop signal, indicating no concrete behavior delta or measurable improvement.d97c067CONFIRMED_SLOP (behavior delta: none): The commit only adds a CLA signature to cla.json, which is purely administrative with no impact on code or product functionality. This aligns with the 'cla_admin' slop signal and does not provide any measurable value to the repository's functionality or quality.eaf5ff2CONFIRMED_SLOP (behavior delta: none): The commit adds a CLA signature to cla.json, which is purely administrative and does not introduce any code or product value. The first-pass verdict correctly identifies this as cla_admin, a category excluded from being demoted to GOOD by the verification bias. There is no behavior delta, and the change does not strengthen or weaken any invariants.f8f7ac1CONFIRMED_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-mergable as per the verification bias rules. No behavior delta exists, and the change does not introduce any new functionality or improvements.54a94d4CONFIRMED_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 change does not alter any runtime behavior, user-visible output, or invariants. It fits the cla_admin category, which is excluded from being demoted to GOOD by the verification bias.81a03faCONFIRMED_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' slop signal and does not provide any concrete behavior delta, abstractions, or user-visible changes. As per the verification bias, cla_admin commits are not considered for demotion to GOOD.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. The first-pass verdict correctly identifies it as SLOP following the bias against release version bumps.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. This aligns with the 'cla_admin' category, which is explicitly non-mergable as per the verification bias. No behavior delta exists, and the change does not improve functionality or correctness.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, indicating administrative noise. This aligns with the 'release_version_bump' category, which is not considered GOOD by default. 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