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 value, including critical bug fixes, documentation improvements, and refactorings with clear benefits. However, the SLOP ratio (23.2%) indicates a substantial amount of non-value or low-value changes, including administrative noise (CLA signatures), minor visibility churn, and refactorings without clear payoff. While the GOOD commits outweigh the SLOP, the presence of administrative noise and minor churn suggests a targeted approach. No critical breaking changes, dependency conflicts, or architecture drift were identified that would outright prevent merging.
Batch classification
Verdict
Count
Branch
🟢 GOOD
88
sync/upstream/v3.17.4-good
🟡 NEEDS_REVIEW
1
sync/upstream/v3.17.4-needs-review
🔴 SLOP
27
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 meaningful functionality with clear user value, including new tests and logical agent resolution logic.
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 maintenance benefit as it aligns with the suggested idiomatic approach for bun:test.
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.
e9b6bba1refactor(agent-display): centralize name normalization resolution -- The commit centralizes name normalization for agent displays, adds meaningful tests for legacy name conversion and UI prefix stripping, and refactors the resolution logic for clarity and reuse.
f95e7a6crefactor(openclaw): reuse shared gateway url validation -- The commit refactors the code to reuse shared gateway URL validation by importing and using validateGatewayUrl from ./config, removing redundant local implementation.
40411f32docs(agents): add AGENTS.md documentation for prometheus, hephaestus, sisyphus variants, and builtin-skills -- Added substantial documentation for multiple agents and 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, including HTTPS, local HTTP, and invalid URLs.
a2ae3de3refactor(openclaw): extract gateway url validator -- Extracted a reusable function validateGatewayUrl into a separate file, reducing code duplication and improving maintainability.
69f1c9aefix(doctor): add timeouts to subprocess spawns to prevent exit code 137 -- Adds 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, directly addressing a specific functionality gap.
d21ed3f7fix(doctor): add timeouts to subprocess spawns to prevent exit code 137 -- Adds timeouts to prevent subprocess hangs, directly addressing 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 specific compatibility problem with macOS Gatekeeper.
71b5ab67fix(ci): replace breaking codesign step with signature verification -- The commit fixes a concrete CI issue by replacing a breaking codesign step with signature verification for macOS binaries, directly addressing a compatibility problem.
16e7fa5bfix(ci): ad-hoc sign darwin binaries with plain codesign -- The commit fixes a concrete CI issue by explicitly signing Darwin binaries with 'codesign --sign -' to address a compatibility problem with macOS Gatekeeper, as Bun on GitHub macOS runners does not emit linker-signed signatures by default.
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 compatibility problem.
2d65896brefactor(shared): simplify normalize SDK null guards -- Simplifies null checks in normalizeSDKResponse from two separate checks to a single '== null' check, improving readability without altering functionality.
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 altering functionality.
cd8352c1refactor(claude-code-mcp-loader): simplify env expansion null guard -- Simplifies null checks in env expansion from two separate checks to a single '== null' check, improving readability without altering functionality.
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 config file parsing 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.
4de02094test(background-agent): lock nullish loop detector behavior -- Adds meaningful test coverage for nullish input handling in loop detector behavior, ensuring consistent handling of 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 altering functionality.
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 a regex to avoid false positives with npm scoped packages, adding a specific test case to validate the change.
3d2eb6e4fix(config): preserve explicit git_master overrides during merge -- Adds explicit test cases and logic to preserve user git_master config overrides, addressing a concrete behavior issue with clear functional value.
dac0c99efeat: add Vercel AI Gateway as a provider -- Adds Vercel AI Gateway as a recognized provider with clear functional value, including config detection, validation, and fallback logic integration.
aac17135test: add vercel provider transform tests -- Adds meaningful test coverage for Vercel provider transformations with multiple scenarios
6f72ec75test: add integration tests for Vercel AI Gateway model resolution -- Adds integration tests for Vercel AI Gateway model resolution with specific scenarios and assertions, increasing test coverage for a real feature.
542dc890fix(provider): use gateway-specific model IDs for vercel transform -- The commit fixes a concrete behavior issue by using gateway-specific model IDs for Vercel transform, with clear functional value and test coverage adjustments.
34e334eafeat(provider): add vercel to all gateway-supported fallback entries -- Adds vercel to all gateway-supported fallback entries with clear functional value and test updates
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.
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 impact and test coverage adjustments.
e3b5c2befix: address review feedback for Vercel AI Gateway PR -- Adds Vercel AI Gateway integration with CLI option, priority adjustment, and test updates, providing clear functional 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 value and code changes that directly address the issue.
3673b962docs(readme-ru): add telemetry and legal section translation -- Added meaningful Russian translations for telemetry and legal sections in README.ru.md with clear user value and functional information.
23d0a0dcdocs(readme-zh-cn): add telemetry and legal section translation -- Added meaningful Chinese translations for telemetry and legal sections in README.zh-cn.md with clear user value and functional information.
0ece66e3docs(readme-ja): add telemetry and legal section translation -- Added meaningful Japanese translations for telemetry and legal sections in README.ja.md with clear user value and functional information.
12ed9501docs(readme-ko): add telemetry and legal section translation -- Added meaningful Korean translations for telemetry and legal sections in README.ko.md with clear user value and functional information.
c91db2ebdocs(cli): add --vercel-ai-gateway flag documentation -- Added documentation for a new CLI flag with clear description and default 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 informing users of an additional option.
9342e55bdocs(installation): replace npx with bunx and update provider flags -- Updated installation docs to reflect 'bunx' instead of 'npx' and added new provider flags with clear explanations.
05e54444docs(overview): update model-role claims and complete category list -- Updated documentation with concrete, substantive information about model roles and categories, enhancing user understanding and configuration clarity.
7c94ee45docs(agent-model-matching): sync fallback chains with current source -- Updated documentation to reflect accurate fallback chains with clear provider paths (e.g., adding 'vercel' to model IDs) for user clarity and configuration accuracy.
b57a7975docs(ollama): verify troubleshooting steps and issue links -- Updated troubleshooting steps with specific issue links and clear documentation of a workaround, adding concrete user value.
bbbd1bd0docs(orchestration): update diagram model names and category tables -- The commit updates documentation with concrete, substantive changes to model names, category tables, and fallback chains, enhancing user understanding and configuration clarity.
4ce1dde3CLI: isolate provider model id transforms from shared mocks -- Added multiple meaningful tests for provider model ID transformations across various providers (github-copilot, google, anthropic, vercel, etc.) with clear expected outputs, significantly increasing test coverage for real scenarios.
1b36194bfix(cli): treat missing session status as idle in run completion -- Added test case for handling missing session status as idle, ensuring pollForCompletion behaves correctly in edge cases.
b5250555fix(telemetry): guard PostHog init failures -- Adds meaningful test coverage for PostHog init failure scenarios, ensuring no-op client behavior when construction throws.
62c60ae9fix: numeric skill names, ultrawork missing run_in_background, ZWSP agent lookups -- The commit fixes concrete bugs and improves functionality with clear value (e.g., coercing numeric skill names to strings, adding required 'run_in_background' parameters, and stripping ZWSP for agent name comparisons).
76c5356atest(agent-config): add regression tests for agent merge priority order -- Adds comprehensive regression tests for agent merge priority order with clear, real-world scenarios.
da91c535fix(call-omo-agent): address cubic review findings and add requirement-based tests -- The commit fixes concrete issues, adds requirement-based 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 value.
5755a90cfeat(agents): add agent definitions file loader and opencode.json reader -- Adds agent definitions file loader, opencode.json reader, and comprehensive test coverage with clear functional value and real-world utility.
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 value, adding new capabilities and comprehensive test coverage.
3d0fb22cfix(call-omo-agent): adapt dev's model override tests to use createMockCtx factory -- The commit fixes a concrete compatibility issue by adapting tests to use a new factory function, ensuring test validity after a deliberate architectural change.
42445f51fix(agents): address cubic review findings on agent loader -- The commit fixes concrete issues with agent loading, including case-insensitive file extension handling, relative path resolution, and precedence logic, with clear functional improvements and new tests.
cf4b2315fix(agents): replace 'in' with Object.hasOwn() for prototype-safe property checks -- Replaced 'in' with Object.hasOwn() for prototype-safe property checks in opencode-config-agents-reader.ts, addressing a specific issue with inherited properties.
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 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, ensuring correct agent naming in the CLI.
0764526afix(posthog): disable exception autocapture to stay within free tier -- Disables exception autocapture to stay within the 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 comprehensive tests covering immediate resolution, delayed resolution, 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 improvements.
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 architectural improvement with broad impact across the codebase.
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.
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 (local only; not 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, fitting the 'minor_visibility_churn' category with no apparent payoff.
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 functionality or value beyond metadata churn.
6713b30crefactor(hooks): drop no-op directory injector callbacks -- Removes unused 'tool.execute.before' callback with no functional impact, classified as minor churn.
21cad26cdocs(config): remove redundant schema default comments -- Removed redundant schema default comments, which is a minor documentation change with no functional impact.
678011c7docs(contributing): update tool/hook counts and version references -- The commit updates counts and version references in the documentation without adding new substantive information or functionality.
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 examples or clear user value, characteristic of AI-generated fluff.
0cb0a6c0docs(contributing): update tool/hook counts and version references -- Updated counts in documentation without adding new substantive information or functionality.
9aeb87c2chore: bump version to 3.17.3 -- The commit only updates the version number in package.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, which is administrative noise with no code or product value.
c9350c67release: v3.17.4 -- The commit only updates version numbers in multiple package.json files, which is administrative noise with no code or product value.
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 fits the 'minor_visibility_churn' category with no apparent payoff. No runtime behavior changes, and the abstraction's scope reduction lacks justification. A senior engineer would likely question the value of this change without additional context or future plans explicitly mentioned.
9d89bbb CONFIRMED_SLOP (behavior delta: none): The commit changes the visibility of LoadBuiltinCommandsOptions from export to internal without clear functional or architectural benefit, aligning with minor_visibility_churn. No runtime behavior delta exists, and the change lacks a concrete benefit or growth path justification. A senior engineer would likely question the value of this change without further context.
0c5cd3a CONFIRMED_SLOP (behavior delta: none): The commit updates generation dates in multiple AGENTS.md files without adding new functionality or value beyond metadata churn, which aligns with the 'date_count_metadata_churn' slop signal. There is no concrete behavior delta, and the change does not introduce new functionality or fix an issue.
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 change does not affect any runtime behavior, user-visible output, or invariants. It aligns with documented slop patterns for cla_admin.
6713b30 CONFIRMED_SLOP (behavior delta: none): The commit removes unused 'tool.execute.before' callbacks in two hook files, which the first-pass model 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 of no-op code, but this constitutes minor churn without clear value addition. No concrete benefit or behavior change justifies demoting from SLOP.
21cad26 CONFIRMED_SLOP (behavior delta: none): The commit removes redundant schema default comments, which is a minor documentation change with no functional impact. The first-pass reason correctly identifies this as docs_churn, and there's no concrete behavior delta or clear value addition beyond cosmetic changes. A senior engineer would likely question the necessity of this change in a code review, as it doesn't strengthen invariants, change runtime behavior, or add measurable value.
6a0d515 CONFIRMED_SLOP (behavior delta: none): The commit adds only 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. No behavior delta exists as this change does not affect runtime code or user-visible output.
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 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 impact runtime code or user-visible output.
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-mergable as per the verification bias. There's no behavior delta, no new functionality, and the change is purely administrative.
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. Administrative noise such as CLA signatures, while necessary for governance, do not add merge value to the codebase itself and are classified as SLOP by the rubric provided.
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. This aligns with the 'cla_admin' slop signal and does not provide any runtime behavior delta, new functionality, or tangible improvement. The first-pass verdict correctly identifies it as SLOP.
678011c CONFIRMED_SLOP (behavior delta: none): The commit updates counts and version references in the documentation without adding new substantive information or functionality. These changes are considered metadata-only doc edits, which align with documented slop patterns. No runtime code paths change, and the commit message's accuracy is confirmed as it reflects the exact nature of the updates (e.g., "update tool/hook counts and version references"). A senior engineer would likely view this as administrative noise rather than a valuable change, as it does not enhance the project's functionality, readability, or usability in a tangible way.
cfd152a CONFIRMED_SLOP (behavior delta: none): The commit adds two features with vague descriptions lacking concrete examples or measurable benefits. The descriptions for 'review-work' and 'ai-slop-remover' are overly broad and characteristic of AI-generated fluff, aligning with the first-pass slop signals. No concrete behavior delta or user-visible output change is evident from the doc updates alone.
0cb0a6c CONFIRMED_SLOP (behavior delta: none): The commit updates counts in documentation (e.g., changing '10' to '52' in plugin/hook composition) without adding new substantive information or functionality. This aligns with the first-pass reason highlighting metadata-only doc edits and count updates as slop signals. No runtime behavior changes, and the update does not introduce new facts or value beyond numerical corrections.
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. This aligns with the 'cla_admin' category, which is explicitly non-rescueable under the given rubric. No behavior delta exists as it doesn't alter runtime code or user-visible output.
1bb59c3 CONFIRMED_SLOP (behavior delta: none): The commit only updates version numbers in multiple package.json files without any code changes, aligning with the 'release_version_bump' slop pattern. No concrete behavior delta or value addition is present.
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 any measurable value to the repository's merge criteria.
f98f7ec 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.
d97c067 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, abstraction, or user-visible change. As per the verification bias, cla_admin updates are not considered GOOD by default.
eaf5ff2 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, indicating no measurable value for the fork's merge criteria.
f8f7ac1 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.
54a94d4 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. No behavior delta exists as it doesn't affect runtime code or user-visible output.
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 or value to the project's functionality.
9aeb87c CONFIRMED_SLOP (behavior delta: none): The commit only updates the version number in package.json, which is classified as administrative noise (release_version_bump) with no direct code or product value added. The first-pass verdict correctly identifies this as SLOP based on the provided guidelines, which explicitly exclude release version bumps from being considered GOOD. No concrete behavior delta, abstractions, or user-visible changes are introduced.
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 matches the documented slop pattern for cla_admin. No behavior delta exists as it doesn't alter runtime code or user-visible output.
fe091ef CONFIRMED_SLOP (behavior delta: none): The commit only updates the version number in package.json, which is classified as release_version_bump, a documented slop pattern. There is no concrete behavior delta, no added abstractions, and the change is purely administrative. A senior engineer would likely not defend this as valuable in a code review focused on merge value.
c9350c6 CONFIRMED_SLOP (behavior delta: none): The commit only updates version numbers in multiple package.json files, which is administrative noise with no code or product value. This matches the 'release_version_bump' category, which is explicitly non-GOOD by the fork's merge-value rubric. There is no behavior delta, and the change does not strengthen or weaken any invariants beyond version tracking.
Fork-sync impact assessment
Breaking changes
(none detected)
Dependency changes
(none)
Architecture drift
(none)
Hidden concerns
Multiple minor refactorings without clear functional benefits (e.g., 1e2e0019, 9d89bbb0)
Administrative noise (CLA signatures) contributing to SLOP ratio
Action items before merging
Review GOOD refactorings for alignment with fork goals (e.g., 5441c848, e9b6bba1)
Manually verify non-breaking nature of config and agent changes
Assess necessity of updated dependencies in package.json
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: 23%
The release contains a significant number of GOOD commits (88) with tangible value, including critical bug fixes, documentation improvements, and refactorings with clear benefits. However, the SLOP ratio (23.2%) indicates a substantial amount of non-value or low-value changes, including administrative noise (CLA signatures), minor visibility churn, and refactorings without clear payoff. While the GOOD commits outweigh the SLOP, the presence of administrative noise and minor churn suggests a targeted approach. No critical breaking changes, dependency conflicts, or architecture drift were identified that would outright prevent merging.
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 meaningful functionality with clear user value, including new tests and logical agent resolution logic.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 maintenance benefit as it aligns with the suggested idiomatic approach for bun:test.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.e9b6bba1refactor(agent-display): centralize name normalization resolution -- The commit centralizes name normalization for agent displays, adds meaningful tests for legacy name conversion and UI prefix stripping, and refactors the resolution logic for clarity and reuse.f95e7a6crefactor(openclaw): reuse shared gateway url validation -- The commit refactors the code to reuse shared gateway URL validation by importing and usingvalidateGatewayUrlfrom./config, removing redundant local implementation.40411f32docs(agents): add AGENTS.md documentation for prometheus, hephaestus, sisyphus variants, and builtin-skills -- Added substantial documentation for multiple agents and 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, including HTTPS, local HTTP, and invalid URLs.a2ae3de3refactor(openclaw): extract gateway url validator -- Extracted a reusable function validateGatewayUrl into a separate file, reducing code duplication and improving maintainability.69f1c9aefix(doctor): add timeouts to subprocess spawns to prevent exit code 137 -- Adds 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, directly addressing a specific functionality gap.d21ed3f7fix(doctor): add timeouts to subprocess spawns to prevent exit code 137 -- Adds timeouts to prevent subprocess hangs, directly addressing 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 specific compatibility problem with macOS Gatekeeper.71b5ab67fix(ci): replace breaking codesign step with signature verification -- The commit fixes a concrete CI issue by replacing a breaking codesign step with signature verification for macOS binaries, directly addressing a compatibility problem.16e7fa5bfix(ci): ad-hoc sign darwin binaries with plain codesign -- The commit fixes a concrete CI issue by explicitly signing Darwin binaries with 'codesign --sign -' to address a compatibility problem with macOS Gatekeeper, as Bun on GitHub macOS runners does not emit linker-signed signatures by default.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 compatibility problem.2d65896brefactor(shared): simplify normalize SDK null guards -- Simplifies null checks in normalizeSDKResponse from two separate checks to a single '== null' check, improving readability without altering functionality.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 altering functionality.cd8352c1refactor(claude-code-mcp-loader): simplify env expansion null guard -- Simplifies null checks in env expansion from two separate checks to a single '== null' check, improving readability without altering functionality.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 config file parsing 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.4de02094test(background-agent): lock nullish loop detector behavior -- Adds meaningful test coverage for nullish input handling in loop detector behavior, ensuring consistent handling of 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 altering functionality.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 success was incorrectly reported as failure due to a secondary archive operation's status.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 for their 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 updates and config adjustments.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 a regex to avoid false positives with npm scoped packages, adding a specific test case to validate the change.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 /plan command references with Prometheus delegation guidance and correcting a skill name to match an actual built-in skill, with relevant 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) -- Fixes a concrete behavior issue by excluding node_modules from git diff verification, with clear test coverage added.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 syntax usage.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 the doctor's 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) -- 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) -- Fixes a concrete behavior issue by adjusting deduplication logic to prioritize real session.idle events over synthetic ones, with clear test coverage added.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 returned 'Task aborted' prematurely, now fetching session results before honoring the abort signal with clear test coverage added.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 loop for sendSyncPrompt failures using a fallback chain, with clear test coverage added.3d2eb6e4fix(config): preserve explicit git_master overrides during merge -- Adds explicit test cases and logic to preserve user git_master config overrides, addressing a concrete behavior issue with clear functional value.dac0c99efeat: add Vercel AI Gateway as a provider -- Adds Vercel AI Gateway as a recognized provider with clear functional value, including config detection, validation, and fallback logic integration.aac17135test: add vercel provider transform tests -- Adds meaningful test coverage for Vercel provider transformations with multiple scenarios6f72ec75test: add integration tests for Vercel AI Gateway model resolution -- Adds integration tests for Vercel AI Gateway model resolution with specific scenarios and assertions, increasing test coverage for a real feature.542dc890fix(provider): use gateway-specific model IDs for vercel transform -- The commit fixes a concrete behavior issue by using gateway-specific model IDs for Vercel transform, with clear functional value and test coverage adjustments.34e334eafeat(provider): add vercel to all gateway-supported fallback entries -- Adds vercel to all gateway-supported fallback entries with clear functional value and test updateseffae16crefactor(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.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 impact and test coverage adjustments.e3b5c2befix: address review feedback for Vercel AI Gateway PR -- Adds Vercel AI Gateway integration with CLI option, priority adjustment, and test updates, providing clear functional 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 value and code changes that directly address the issue.3673b962docs(readme-ru): add telemetry and legal section translation -- Added meaningful Russian translations for telemetry and legal sections in README.ru.md with clear user value and functional information.23d0a0dcdocs(readme-zh-cn): add telemetry and legal section translation -- Added meaningful Chinese translations for telemetry and legal sections in README.zh-cn.md with clear user value and functional information.0ece66e3docs(readme-ja): add telemetry and legal section translation -- Added meaningful Japanese translations for telemetry and legal sections in README.ja.md with clear user value and functional information.12ed9501docs(readme-ko): add telemetry and legal section translation -- Added meaningful Korean translations for telemetry and legal sections in README.ko.md with clear user value and functional information.c91db2ebdocs(cli): add --vercel-ai-gateway flag documentation -- Added documentation for a new CLI flag with clear description and default 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 informing users of an additional option.9342e55bdocs(installation): replace npx with bunx and update provider flags -- Updated installation docs to reflect 'bunx' instead of 'npx' and added new provider flags with clear explanations.05e54444docs(overview): update model-role claims and complete category list -- Updated documentation with concrete, substantive information about model roles and categories, enhancing user understanding and configuration clarity.7c94ee45docs(agent-model-matching): sync fallback chains with current source -- Updated documentation to reflect accurate fallback chains with clear provider paths (e.g., adding 'vercel' to model IDs) for user clarity and configuration accuracy.b57a7975docs(ollama): verify troubleshooting steps and issue links -- Updated troubleshooting steps with specific issue links and clear documentation of a workaround, adding concrete user value.bbbd1bd0docs(orchestration): update diagram model names and category tables -- The commit updates documentation with concrete, substantive changes to model names, category tables, and fallback chains, enhancing user understanding and configuration clarity.4ce1dde3CLI: isolate provider model id transforms from shared mocks -- Added multiple meaningful tests for provider model ID transformations across various providers (github-copilot, google, anthropic, vercel, etc.) with clear expected outputs, significantly increasing test coverage for real scenarios.1b36194bfix(cli): treat missing session status as idle in run completion -- Added test case for handling missing session status as idle, ensuring pollForCompletion behaves correctly in edge cases.b5250555fix(telemetry): guard PostHog init failures -- Adds meaningful test coverage for PostHog init failure scenarios, ensuring no-op client behavior when construction throws.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 and improves functionality with clear value (e.g., coercing numeric skill names to strings, adding required 'run_in_background' parameters, and stripping ZWSP for agent name comparisons).76c5356atest(agent-config): add regression tests for agent merge priority order -- Adds comprehensive regression tests for agent merge priority order with clear, real-world scenarios.da91c535fix(call-omo-agent): address cubic review findings and add requirement-based tests -- The commit fixes concrete issues, adds requirement-based 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 value.5755a90cfeat(agents): add agent definitions file loader and opencode.json reader -- Adds agent definitions file loader, opencode.json reader, and comprehensive test coverage with clear functional value and real-world utility.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 value, adding new capabilities and comprehensive test coverage.3d0fb22cfix(call-omo-agent): adapt dev's model override tests to use createMockCtx factory -- The commit fixes a concrete compatibility issue by adapting tests to use a new factory function, ensuring test validity after a deliberate architectural change.42445f51fix(agents): address cubic review findings on agent loader -- The commit fixes concrete issues with agent loading, including case-insensitive file extension handling, relative path resolution, and precedence logic, with clear functional improvements and new tests.cf4b2315fix(agents): replace 'in' with Object.hasOwn() for prototype-safe property checks -- Replaced 'in' with Object.hasOwn() for prototype-safe property checks in opencode-config-agents-reader.ts, addressing a specific issue with inherited properties.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 code review findings with concrete fixes, including security patches (prototype pollution prevention), 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 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 regression tests by updating 'code-review' to 'review-work' in test data and adds new tests to prevent false positives in isPlanFamily checks, directly addressing specific issues with clear functional value.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, ensuring correct agent naming in the CLI.0764526afix(posthog): disable exception autocapture to stay within free tier -- Disables exception autocapture to stay within the 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, directly improving functionality with clear user value.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 comprehensive tests covering immediate resolution, delayed resolution, 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 improvements.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 architectural improvement with broad impact across the codebase.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.
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(local only; not 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, fitting the 'minor_visibility_churn' category with no apparent payoff.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 functionality or value beyond 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, classified as minor churn.21cad26cdocs(config): remove redundant schema default comments -- Removed redundant schema default comments, which is a minor documentation change with no functional impact.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 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 -- The 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 documentation without adding new substantive information or functionality.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 examples or clear user value, characteristic of AI-generated fluff.0cb0a6c0docs(contributing): update tool/hook counts and version references -- Updated counts in documentation without adding new substantive 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 -- The commit only 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 -- The 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, which is administrative noise with no code or product value.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, which is administrative noise with no code or product value.c9350c67release: v3.17.4 -- The commit only updates version numbers in multiple package.json files, which is administrative noise with no code or product value.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 fits the 'minor_visibility_churn' category with no apparent payoff. No runtime behavior changes, and the abstraction's scope reduction lacks justification. A senior engineer would likely question the value of this change without additional context or future plans explicitly mentioned.9d89bbbCONFIRMED_SLOP (behavior delta: none): The commit changes the visibility of LoadBuiltinCommandsOptions from export to internal without clear functional or architectural benefit, aligning with minor_visibility_churn. No runtime behavior delta exists, and the change lacks a concrete benefit or growth path justification. A senior engineer would likely question the value of this change without further context.0c5cd3aCONFIRMED_SLOP (behavior delta: none): The commit updates generation dates in multiple AGENTS.md files without adding new functionality or value beyond metadata churn, which aligns with the 'date_count_metadata_churn' slop signal. There is no concrete behavior delta, and the change does not introduce new functionality or fix an issue.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 change does not affect any runtime behavior, user-visible output, or invariants. It aligns with documented slop patterns for cla_admin.6713b30CONFIRMED_SLOP (behavior delta: none): The commit removes unused 'tool.execute.before' callbacks in two hook files, which the first-pass model 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 of no-op code, but this constitutes minor churn without clear value addition. No concrete benefit or behavior change justifies demoting from SLOP.21cad26CONFIRMED_SLOP (behavior delta: none): The commit removes redundant schema default comments, which is a minor documentation change with no functional impact. The first-pass reason correctly identifies this as docs_churn, and there's no concrete behavior delta or clear value addition beyond cosmetic changes. A senior engineer would likely question the necessity of this change in a code review, as it doesn't strengthen invariants, change runtime behavior, or add measurable value.6a0d515CONFIRMED_SLOP (behavior delta: none): The commit adds only 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. No behavior delta exists as this change does not affect runtime code or user-visible output.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 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 impact runtime code or user-visible output.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-mergable as per the verification bias. There's no behavior delta, no new functionality, and the change is purely administrative.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. Administrative noise such as CLA signatures, while necessary for governance, do not add merge value to the codebase itself and are classified as SLOP by the rubric provided.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. This aligns with the 'cla_admin' slop signal and does not provide any runtime behavior delta, new functionality, or tangible improvement. The first-pass verdict correctly identifies it as SLOP.678011cCONFIRMED_SLOP (behavior delta: none): The commit updates counts and version references in the documentation without adding new substantive information or functionality. These changes are considered metadata-only doc edits, which align with documented slop patterns. No runtime code paths change, and the commit message's accuracy is confirmed as it reflects the exact nature of the updates (e.g., "update tool/hook counts and version references"). A senior engineer would likely view this as administrative noise rather than a valuable change, as it does not enhance the project's functionality, readability, or usability in a tangible way.cfd152aCONFIRMED_SLOP (behavior delta: none): The commit adds two features with vague descriptions lacking concrete examples or measurable benefits. The descriptions for 'review-work' and 'ai-slop-remover' are overly broad and characteristic of AI-generated fluff, aligning with the first-pass slop signals. No concrete behavior delta or user-visible output change is evident from the doc updates alone.0cb0a6cCONFIRMED_SLOP (behavior delta: none): The commit updates counts in documentation (e.g., changing '10' to '52' in plugin/hook composition) without adding new substantive information or functionality. This aligns with the first-pass reason highlighting metadata-only doc edits and count updates as slop signals. No runtime behavior changes, and the update does not introduce new facts or value beyond numerical corrections.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. This aligns with the 'cla_admin' category, which is explicitly non-rescueable under the given rubric. No behavior delta exists as it doesn't alter runtime code or user-visible output.1bb59c3CONFIRMED_SLOP (behavior delta: none): The commit only updates version numbers in multiple package.json files without any code changes, aligning with the 'release_version_bump' slop pattern. No concrete behavior delta or value addition is present.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 any measurable value to the repository's merge criteria.f98f7ecCONFIRMED_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.d97c067CONFIRMED_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, abstraction, or user-visible change. As per the verification bias, cla_admin updates are not considered GOOD by default.eaf5ff2CONFIRMED_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, indicating no measurable value for the fork's merge criteria.f8f7ac1CONFIRMED_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.54a94d4CONFIRMED_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. No behavior delta exists as it doesn't affect runtime code or user-visible output.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 or value to the project's functionality.9aeb87cCONFIRMED_SLOP (behavior delta: none): The commit only updates the version number in package.json, which is classified as administrative noise (release_version_bump) with no direct code or product value added. The first-pass verdict correctly identifies this as SLOP based on the provided guidelines, which explicitly exclude release version bumps from being considered GOOD. No concrete behavior delta, abstractions, or user-visible changes are introduced.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 matches the documented slop pattern for cla_admin. No behavior delta exists as it doesn't alter runtime code or user-visible output.fe091efCONFIRMED_SLOP (behavior delta: none): The commit only updates the version number in package.json, which is classified as release_version_bump, a documented slop pattern. There is no concrete behavior delta, no added abstractions, and the change is purely administrative. A senior engineer would likely not defend this as valuable in a code review focused on merge value.c9350c6CONFIRMED_SLOP (behavior delta: none): The commit only updates version numbers in multiple package.json files, which is administrative noise with no code or product value. This matches the 'release_version_bump' category, which is explicitly non-GOOD by the fork's merge-value rubric. There is no behavior delta, and the change does not strengthen or weaken any invariants beyond version tracking.Fork-sync impact assessment
Breaking changes
(none detected)
Dependency changes
(none)
Architecture drift
(none)
Hidden concerns
1e2e0019,9d89bbb0)Action items before merging
GOODrefactorings for alignment with fork goals (e.g.,5441c848,e9b6bba1)package.jsonGenerated 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