[retr0h]

Fixes what #598 left on main. Found while merging the otel 1.47 Dependabot PRs.

What was left behind

#598 bumped go.opentelemetry.io/otel in the root go.mod. Dependabot stops there, and examples/sdk/client has its own go.mod that still required otel 1.46 as an indirect, so main merged untidy. otel 1.47 also introduces otel/log, which enters the indirect block.

Nothing caught it, and nothing can

The CI workflow runs just go-mod in its Deps step, which tidies, before just test runs go-mod-check. The check has nothing left to find. Every green run has meant "CI tidied this for us" rather than "the modules are tidy", and since the tidy is never committed, the untidy state merges anyway.

osapi is the only repository in the organization that does this:

osapi                1
gohai                0
nats-client          0
nats-server          0
osapi-orchestrator   0

That is exactly why the identical otel 1.47 bump failed in osapi-io/nats-client#172 and passed on all four PRs here.

Removing the line makes the check work, verified locally:

untidy: examples/sdk/client
run 'just go-mod' and commit the result

The CI change is not in this PR

It needs one line removed from .github/workflows/go.yml:

         run: |
           just go-deps
-          just go-mod
           just react-deps

I could not push it. Every attempt is rejected with remote: Internal Server Error, and the same branch pushes fine once the workflow file is out of the diff, so the credential in this session is not allowed to modify workflow files.

Safe to remove: just go-deps already provides what the test recipes need, which is how the other four repositories work.

just test passes, coverage 100%.

🤖 Generated with Claude Code

https://claude.ai/code/session_01FuKUsHFG1EqZXamffh9M2c

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

@@           Coverage Diff           @@
##             main     #602   +/-   ##
=======================================
  Coverage   99.95%   99.95%           
=======================================
  Files         501      501           
  Lines       24099    24099           
=======================================
  Hits        24089    24089           
  Misses         10       10           

---

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f0f727f...f2f520f. Read the comment docs.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[retr0h]

Superseded by the combined change: the CI step and the tidy it was hiding belong in one PR. My reason for splitting them was wrong — I read three transient GitHub 500s as a rule against pushing workflow files, and the push works fine.