-
Notifications
You must be signed in to change notification settings - Fork 15
Comparing changes
Open a pull request
base repository: hookdeck/hookdeck-cli
base: fc494f0
head repository: hookdeck/hookdeck-cli
compare: 09f261c
- 12 commits
- 14 files changed
- 2 contributors
Commits on Nov 11, 2025
-
Configuration menu - View commit details
-
Copy full SHA for 397a6f6 - Browse repository at this point
Copy the full SHA 397a6f6View commit details
Commits on Nov 14, 2025
-
feat: add --local flag to project use command
Add support for pinning Hookdeck project configuration to the current directory using the --local flag. This enables per-repository project configuration for better developer experience in multi-project workflows. Implementation: - Added --local flag to 'project use' command - Implemented UseProjectLocal() method to create/update local config - Added smart default: auto-updates local config when it exists - Validates --local and --config are mutually exclusive - Displays security warning when creating local config - Refactored config writing to reduce code duplication Testing: - Created comprehensive acceptance test suite - Added helper functions for temp directory management - Tests validate flag behavior, config creation, and security warnings - 2 tests passing in CI, 4 tests ready for CLI key testing Documentation: - Updated README with complete --local flag documentation - Explained configuration file precedence (--config > local > global) - Added security guidance for credential management - Included examples for all use cases Related: Previous PRs #102, #103 that removed --local flag
Configuration menu - View commit details
-
Copy full SHA for a81a219 - Browse repository at this point
Copy the full SHA a81a219View commit details -
feat: add manual test workflow for CLI authentication
Add semi-automated testing workflow for project use tests that require human browser-based authentication. These tests ensure the --local flag works correctly with CLI key authentication. Implementation: - Created project_use_manual_test.go with 4 manual tests (build tag: manual) - Added RequireCLIAuthentication() helper that forces fresh login - Helper clears existing auth, runs 'hookdeck login', waits for user - Existing automated tests remain in project_use_test.go (run in CI) - Manual tests only run with: go test -tags=manual Testing workflow: - Tests clear authentication and require fresh CLI login - User completes browser authentication when prompted - Tests verify local config creation with project selection - Validates security warnings and smart default behavior Documentation: - Updated test/acceptance/README.md with manual test instructions - Explained difference between automated (CI) and manual tests - Provided examples of running manual tests locally - Documented which tests require human authentication Feature implementation: - Added security warning output in project_use.go for new local configs - Fixed config writing logic in config.go to properly handle new files - Refactored project_use_test.go to separate automated vs manual tests - Updated test comments to clarify CI vs local execution requirements Related: Completes semi-automated testing for --local flag feature
Configuration menu - View commit details
-
Copy full SHA for a7eed65 - Browse repository at this point
Copy the full SHA a7eed65View commit details -
chore: display source and destination types in connection output
- Add type display in brackets [TYPE] for connection list view - Add 'Type:' field to connection get detailed view - Update tests to verify type fields are displayed correctly
Configuration menu - View commit details
-
Copy full SHA for 6201a18 - Browse repository at this point
Copy the full SHA 6201a18View commit details -
feat: add name-based lookup and improve error handling for connection…
… get - Add support for passing connection name to 'connection get' command - Implement resolveConnectionID() to accept both ID and name - Improve 404 error handling with user-friendly messages - Add tests for name-based lookup and error cases - Update command usage and help text to reflect name support
Configuration menu - View commit details
-
Copy full SHA for 77c4831 - Browse repository at this point
Copy the full SHA 77c4831View commit details -
Configuration menu - View commit details
-
Copy full SHA for e984709 - Browse repository at this point
Copy the full SHA e984709View commit details
Commits on Nov 21, 2025
-
chore(docs): add instructions for force linking Homebrew installation…
… of hookdeck-beta
Configuration menu - View commit details
-
Copy full SHA for f6c857a - Browse repository at this point
Copy the full SHA f6c857aView commit details -
Merge branch 'feat/local-flag' of https://github.com/hookdeck/hookdec…
…k-cli into feat/local-flag
Configuration menu - View commit details
-
Copy full SHA for 867bf9d - Browse repository at this point
Copy the full SHA 867bf9dView commit details -
fix: connection upsert with only destination config flags now works
- Added destinationURL and destinationCliPath to hasAnyDestinationFlag() check - Added destinationURL and destinationCliPath to hasDestinationConfigOnly check - Added destinationURL and destinationCliPath to hasDestinationConfigUpdate check - Added destinationCliPath handling in buildDestinationInputForUpdate() - Added comprehensive test suite for partial updates (TestConnectionUpsertPartialUpdates) This fixes the 422 error when running: hookdeck connection upsert <name> --destination-url <URL> Previously, the CLI wasn't detecting that destination config flags were provided, so it didn't fetch the existing connection to preserve source/destination references. The API requires both source and destination in every PUT request, so the CLI now automatically handles this by fetching the existing connection and merging changes.
Configuration menu - View commit details
-
Copy full SHA for f06659a - Browse repository at this point
Copy the full SHA f06659aView commit details -
Merge pull request #175 from hookdeck/fix/conn-upsert
fix: connection upsert with only destination config flags now works
Configuration menu - View commit details
-
Copy full SHA for 9089183 - Browse repository at this point
Copy the full SHA 9089183View commit details -
Merge pull request #174 from hookdeck/feat/local-flag
Project Use --local Flag & Connection Management Improvements
Configuration menu - View commit details
-
Copy full SHA for b9d3312 - Browse repository at this point
Copy the full SHA b9d3312View commit details -
Configuration menu - View commit details
-
Copy full SHA for 09f261c - Browse repository at this point
Copy the full SHA 09f261cView commit details
This comparison is taking too long to generate.
Unfortunately it looks like we can’t render this comparison for you right now. It might be too big, or there might be something weird with your repository.
You can try running this command locally to see the comparison on your machine:
git diff fc494f0...09f261c