Skip to content

[FAILED] sdk-sync: API 2.0.1+0000001 - #95

Closed
marketdata-sdk-sync[bot] wants to merge 1 commit into
mainfrom
sdk-sync/2.0.1-0000001
Closed

marketdata-sdk-sync[bot] wants to merge 1 commit into
mainfrom
sdk-sync/2.0.1-0000001

Conversation

@marketdata-sdk-sync

@marketdata-sdk-sync marketdata-sdk-sync Bot commented Oct 14, 2026 •

Copy link
Copy Markdown

Failed. The pipeline could not finish this change: build failed on the tests-only change: Execution failed for task ':spotlessJavaCheck' (registered in build file 'build.gradle.kts').. This draft is what the coder produced before the pipeline stopped; nothing in it passed the gates. The full story is in the job log comment on this pull request and in the failures/ folder of the state repository.
Generated by sdk-auto-sync for API release 2.0.1 (commit 0000001).

Changes in the API surface

Snapshots c2a31e277345 → 8679cb068959:

- GET /v1/markets/status/ ?countback: integer

What the agent did

tests: The SDK already fully implemented countback for markets/status (MarketStatusRequest, MarketRequests.validateWindow, MarketsResource.statusSpec) with wire, decode, and validation tests. I closed two coverage gaps: a unit test asserting date+countback are rejected as mutually exclusive (mirroring the equivalent OptionsResourceTest case, previously untested for markets), and a stronger integration test asserting countback returns exactly 5 days and widens the result past the unfiltered single-day default, replacing the existing weak isNotEmpty()-only assertion (left untouched, per instructions, only added to).

Files

  • src/integrationTest/java/com/marketdata/sdk/MarketsIntegrationTest.java
  • src/test/java/com/marketdata/sdk/MarketsResourceTest.java

Live-API suite

  • src/integrationTest/java/com/marketdata/sdk/MarketsIntegrationTest.java

Files of this suite changed by this pull request. The pipeline did not run the suite (it has no API token); this pull request's CI does.

Job log

The full record of this job — every phase, test and build result, review findings and how they were resolved, and each fix round after the PR was opened — is in the job log comment.


1 coder run(s) · 0 review(s) · 21 turns · 1,326,563 tokens · US$ 0.75 · item a0c778e1-1561-4ec4-b3e3-14803f13ad1c

Agent runs
  • coder · claude-haiku-4-5-20251001, claude-sonnet-5 · 1,326,563 tokens (88,360 in, 1,223,732 cached, 14,471 out) · 21 turns · US$ 0.75 · 2 min 56 s

tests: The SDK already fully implemented `countback` for `markets/status` (MarketStatusRequest, MarketRequests.validateWindow, MarketsResource.statusSpec) with wire, decode, and validation tests. I closed two coverage gaps: a unit test asserting `date`+`countback` are rejected as mutually exclusive (mirroring the equivalent OptionsResourceTest case, previously untested for markets), and a stronger integration test asserting countback returns exactly 5 days and widens the result past the unfiltered single-day default, replacing the existing weak `isNotEmpty()`-only assertion (left untouched, per instructions, only added to).
@marketdata-sdk-sync marketdata-sdk-sync Bot added the sdk-sync: needs human sdk-sync stopped: a person has to look (see the PR's comments) label Oct 14, 2026
@marketdata-sdk-sync

Copy link
Copy Markdown
Author

🧾 Job log — sdk-sync

sdk-auto-sync job log

API release 2.0.1 (commit 0000001)
SDK sdk-java
Branch sdk-sync/2.0.1-0000001
API surface c2a31e277345 → 8679cb068959
Date 2026-10-14 14:06:29Z
Outcome failed — build failed on the tests-only change: Execution failed for task ':spotlessJavaCheck' (registered in build file 'build.gradle.kts').

Changes in the API surface

- GET /v1/markets/status/ ?countback: integer

1. tests

Coder (21 turns, US$ 0.75, 2 min 56 s):

gradle test --no-daemon --console=plain ran once: BUILD SUCCESSFUL (green), not red. No production code needed the countback behavior added — it already exists and the new tests pass against it, so there was no missing symbol to cause a compile failure.

The SDK already fully implemented countback for markets/status (MarketStatusRequest, MarketRequests.validateWindow, MarketsResource.statusSpec) with wire, decode, and validation tests. I closed two coverage gaps: a unit test asserting date+countback are rejected as mutually exclusive (mirroring the equivalent OptionsResourceTest case, previously untested for markets), and a stronger integration test asserting countback returns exactly 5 days and widens the result past the unfiltered single-day default, replacing the existing weak isNotEmpty()-only assertion (left untouched, per instructions, only added to).

Gates:

  • tests (expected to fail), attempt 1: passed (7 s)

2. build

Gates:

  • build, attempt 1: failed — Execution failed for task ':spotlessJavaCheck' (registered in build file 'build.gradle.kts'). (11 s)

Totals

  • Bump: not assessed
  • Coder runs: 1
  • Review runs: 0
  • Gates run: 2 (1 failed)
  • Files changed by the agent: 2
  • Turns: 21
  • Tokens: 1,326,563
  • Estimated cost: US$ 0.75
  • Time: 3 min 29 s (agents 2 min 56 s, gates 18 s)

Agent runs

  • coder · claude-haiku-4-5-20251001, claude-sonnet-5 · 1,326,563 tokens (88,360 in, 1,223,732 cached, 14,471 out) · 21 turns · US$ 0.75 · 2 min 56 s

@MarketDataDev03

Copy link
Copy Markdown
Collaborator

Test run of the major path; closing.

@MarketDataDev03
MarketDataDev03 deleted the sdk-sync/2.0.1-0000001 branch October 14, 2026 14:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

sdk-sync: needs human sdk-sync stopped: a person has to look (see the PR's comments)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant