Skip to content

Upgrade quant-client to v4.12.0, fix env logs, and extend tests#8

Merged
stooit merged 1 commit into
mainfrom
feat/client-upgrade-and-fixes
Apr 5, 2026
Merged

Upgrade quant-client to v4.12.0, fix env logs, and extend tests#8
stooit merged 1 commit into
mainfrom
feat/client-upgrade-and-fixes

Conversation

@stooit
Copy link
Copy Markdown
Contributor

@stooit stooit commented Mar 9, 2026

Summary

  • Upgrade @quantcdn/quant-client from 4.3.0 to 4.12.0 (no breaking changes for current CLI usage; crawlersList void return type reported upstream)
  • Fix qc env logs — was returning no data because the API response uses logEvents key, not logs
  • Simplify API client — remove redundant manual headers, rely on Configuration.accessToken
  • Consistent machine_name usage — project and crawler selection now always uses machine_name for API calls instead of falling back to display name
  • Autocomplete prompts — crawler command now uses searchable autocomplete for project/crawler selection
  • Cleaner CLI banner — replace gradient figlet banner with minimal ASCII art in cyan
  • Expanded test suite — API client tests from 14 to 52, covering all methods with mocked SDK calls (happy paths, parameter overrides, error handling for 401/403/404)

Test plan

  • All 86 unit tests pass
  • Build succeeds with no type errors
  • qc env logs returns log data
  • Banner displays correctly
  • Manual test of qc crawler run autocomplete prompts
  • Manual test of qc project select with machine_name

- Upgrade @quantcdn/quant-client from 4.3.0 to 4.12.0
- Fix env logs returning no data (response uses logEvents, not logs)
- Simplify API client config to use Configuration.accessToken
- Use machine_name consistently for project/crawler selection
- Add autocomplete prompts for crawler project/crawler selection
- Add DOM lib to tsconfig for VRT type annotations
- Expand explicit params for backup and metrics API calls
- Replace ASCII banner with cleaner logo in cyan
- Extend API client test suite from 14 to 52 tests with mocked SDK
@stooit stooit merged commit 2cdf43b into main Apr 5, 2026
8 checks passed
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.

2 participants