Skip to content

Fix staging sync base URL#38

Closed
ZhimaoL wants to merge 1 commit into
asksurf-ai:stgfrom
ZhimaoL:stg-base-url-sync-fix
Closed

Fix staging sync base URL#38
ZhimaoL wants to merge 1 commit into
asksurf-ai:stgfrom
ZhimaoL:stg-base-url-sync-fix

Conversation

@ZhimaoL

@ZhimaoL ZhimaoL commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Summary

  • cherry-pick [FIX] sync surf API spec from configured base #27 onto stg so surf sync respects SURF_API_BASE_URL / --surf-api-base-url
  • keep the PR scoped to the base URL sync fix only; no main-to-stg merge
  • unblock staging CLI/skill e2e against Hermod exchange endpoints

Verification

  • HOME=/tmp/surf-cli-stg-baseurl-test-home go test ./...
  • go build -o /tmp/surf-stg-baseurl ./cmd/surf
  • HOME=/tmp/surf-stg-baseurl-home SURF_API_BASE_URL=https://api.stg.ask.surf/gateway/v1 /tmp/surf-stg-baseurl sync
  • /tmp/surf-stg-baseurl list-operations -g | rg "exchange-(coverage|candles)"
  • /tmp/surf-stg-baseurl exchange-coverage --exchange bithumb --limit 2 --json
  • /tmp/surf-stg-baseurl exchange-candles --exchange bithumb --pair BTC/KRW --interval 5m --limit 2 --json

* [FIX] sync surf API spec from configured base

* fix(tests): build surf binary with windows extension

* fix(cli): keep surf cache tests hermetic

---------

Co-authored-by: HappySean <zhaoxiang2845@gmail.com>
@ZhimaoL

ZhimaoL commented Jun 24, 2026

Copy link
Copy Markdown
Contributor Author

Closing this because it is a generic staging sync-base-url fix, not the exchange endpoint skill/CLI integration work. Keeping the exchange task focused on OpenAPI-driven CLI discovery and surf skill guidance.

@ZhimaoL ZhimaoL closed this Jun 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant