Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: hookdeck/hookdeck-cli
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: fc494f0
Choose a base ref
...
head repository: hookdeck/hookdeck-cli
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 09f261c
Choose a head ref
  • 12 commits
  • 14 files changed
  • 2 contributors

Commits on Nov 11, 2025

  1. Configuration menu
    Copy the full SHA
    397a6f6 View commit details
    Browse the repository at this point in the history

Commits on Nov 14, 2025

  1. 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
    leggetter committed Nov 14, 2025
    Configuration menu
    Copy the full SHA
    a81a219 View commit details
    Browse the repository at this point in the history
  2. 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
    leggetter committed Nov 14, 2025
    Configuration menu
    Copy the full SHA
    a7eed65 View commit details
    Browse the repository at this point in the history
  3. 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
    leggetter committed Nov 14, 2025
    Configuration menu
    Copy the full SHA
    6201a18 View commit details
    Browse the repository at this point in the history
  4. 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
    leggetter committed Nov 14, 2025
    Configuration menu
    Copy the full SHA
    77c4831 View commit details
    Browse the repository at this point in the history
  5. Configuration menu
    Copy the full SHA
    e984709 View commit details
    Browse the repository at this point in the history

Commits on Nov 21, 2025

  1. Configuration menu
    Copy the full SHA
    f6c857a View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    867bf9d View commit details
    Browse the repository at this point in the history
  3. 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.
    leggetter committed Nov 21, 2025
    Configuration menu
    Copy the full SHA
    f06659a View commit details
    Browse the repository at this point in the history
  4. Merge pull request #175 from hookdeck/fix/conn-upsert

    fix: connection upsert with only destination config flags now works
    leggetter authored Nov 21, 2025
    Configuration menu
    Copy the full SHA
    9089183 View commit details
    Browse the repository at this point in the history
  5. Merge pull request #174 from hookdeck/feat/local-flag

    Project Use --local Flag & Connection Management Improvements
    leggetter authored Nov 21, 2025
    Configuration menu
    Copy the full SHA
    b9d3312 View commit details
    Browse the repository at this point in the history
  6. Merge pull request #176 from hookdeck/main

    Main
    leggetter authored Nov 21, 2025
    Configuration menu
    Copy the full SHA
    09f261c View commit details
    Browse the repository at this point in the history
Loading