Add query command for running ad-hoc queries#16
Closed
maragubot wants to merge 6 commits into
Closed
Conversation
- Update module path to github.com/maragudk/honeycomb-cli - Add cobra CLI framework with root command and global flags - Support --api-key/HONEYCOMB_API_KEY and --api-url/HONEYCOMB_API_URL - Add version command (set via -ldflags at build time) - Add basic test for Execute function - Update README with install and usage instructions Closes #1
- Create honeycomb package with HTTP client supporting API key auth - Add error response parsing (RFC 7807 Problem Detail format) - Implement GET /1/auth endpoint for key verification - Add `auth` command showing team, environment, and permissions - Refactor commands to use NewRootCommand() for testability - Add tests for client, auth API, and auth command Closes #2 Closes #3
- Add Dataset types and API methods to honeycomb package - Add datasets subcommands with table and JSON output - Add newClient helper for creating API clients from flags - Tests for all API methods and commands Closes #4
- Add Marker types and API methods for list, create, delete - Support --dataset flag (defaults to __all__ for environment-wide) - Markers are useful for annotating deploys in CI/CD pipelines - Table and JSON output formatting - Tests for all API methods and commands Closes #5
- Add Column types and ListColumns API method - Add columns list command with --dataset flag (required) - Table and JSON output formatting - Tests for API method and command Closes #6
- Add query spec types, create/execute/poll API methods - Add RunQuery convenience method that handles full query lifecycle - Add query command with --calculation, --breakdown, --filter, --time-range flags - Support multiple calculations (e.g. COUNT, AVG:column, P99:column) - Support filter parsing (e.g. "status_code = 200") - Table and JSON output formatting - Tests for API methods, calculation/filter parsing, and command output Closes #7
maragubot
enabled auto-merge (squash)
February 27, 2026 12:44
Collaborator
Author
|
Changes included in PR #18 |
auto-merge was automatically disabled
February 27, 2026 12:50
Pull request was closed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
querycommand with flags for calculations, breakdowns, filters, and time rangeCOUNT,AVG:duration_ms,P99:latencystatus_code = 200,name contains fooThis is the highest-value CLI feature, enabling ad-hoc querying from the terminal.
Test plan
go install .worksCloses #7