Skip to content
This repository was archived by the owner on Aug 21, 2025. It is now read-only.

chore: update prefix#114

Merged
kristencheung merged 1 commit into
mainfrom
update-prefix
May 5, 2025
Merged

chore: update prefix#114
kristencheung merged 1 commit into
mainfrom
update-prefix

Conversation

@kristencheung
Copy link
Copy Markdown
Contributor

@kristencheung kristencheung commented May 2, 2025

Description

Update prefix to accommodate different types of api keys

Dependent on migration
https://github.com/hypermodeinc/hypermode/pull/425

Checklist

  • Code compiles correctly and linting passes locally
  • Tests added for new functionality, or regression tests for bug fixes added as applicable
  • For new features, etc., PR on docs repo staged and
    linked here

@kristencheung kristencheung marked this pull request as ready for review May 5, 2025 14:21
@kristencheung kristencheung requested review from a team and Copilot May 5, 2025 14:21
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR updates the API key prefix in the login command to better accommodate different types of API keys and bumps several dependency versions in the trunk configuration file.

  • Update prefix from 'cli' to 'hyp_usr' and include metadata (ownerType and ownerId) for the API key.
  • Bump CLI and plugin versions in the trunk YAML configuration.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
src/commands/login/index.ts Changed API key prefix and added metadata fields for key details.
.trunk/trunk.yaml Updated dependency versions to meet migration requirements.
Comments suppressed due to low confidence (2)

src/commands/login/index.ts:144

  • The API key prefix was changed from 'cli' to 'hyp_usr' and metadata was added; please confirm that the downstream processing logic aligns with the new structure and that this change meets the overall migration requirements.
prefix: 'hyp_usr',

.trunk/trunk.yaml:5

  • Ensure that bumping the CLI version to 1.22.15 is fully compatible with the migration requirements and that any accompanying notes or documentation are updated accordingly.
version: 1.22.15

@kristencheung kristencheung requested a review from octref May 5, 2025 14:28
@kristencheung kristencheung merged commit 0491921 into main May 5, 2025
6 checks passed
@kristencheung kristencheung deleted the update-prefix branch May 5, 2025 16:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants