fix: rename 'Loki' → 'Lowkey' prefix + hermes native Bedrock#76
Conversation
Account rename: - Default prefix changed from 'Loki-' to 'Lowkey-' - _account_already_prefixed checks for 'lowkey*' prefix - Strip length updated from 5 to 7 chars Hermes pack (v2.0.0): - Remove bedrockify dependency (native Bedrock via Converse API) - Pin hermes to v2026.7.7.2 (commit b7751df) for reproducibility - provider: bedrock (not 'custom' via proxy) - Install [bedrock] extras (boto3) via uv - No OPENAI_API_KEY needed — IAM instance role auth - Default model: us.anthropic.claude-sonnet-4-6 - Tested locally: hermes -z works with native Bedrock
royosherove
force-pushed
the
fix/lowkey-rename-hermes-native-bedrock
branch
from
5c2a03f to
8774397
Compare
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 5c2a03f243
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
|
|
||
| deps: | ||
| - bedrockify | ||
| deps: [] |
There was a problem hiding this comment.
For CloudFormation/Terraform deployments I checked deploy/bootstrap.sh, and it resolves dependencies with registry_get_deps from packs/registry.yaml, not from this manifest; packs/registry.yaml/.json still declare hermes depends on bedrockify. As a result, --pack hermes still installs packs/bedrockify/install.sh before the native-Bedrock Hermes pack, so deployments can still fail or spend time on an unneeded proxy even though this manifest says there are no deps. Please keep the registry in sync with this manifest change.
Useful? React with 👍 / 👎.
1 participant
Changes
1. Account rename default: Loki → Lowkey
Loki-toLowkey-_account_already_prefixednow checks forlowkey*2. Hermes pack: native Bedrock (no bedrockify)
Hermes 0.18.2 supports Bedrock natively via the Converse API (docs).
Before:
deps: [bedrockify],provider: custom, OpenAI-compat proxy on :8090After:
deps: [],provider: bedrock, direct IAM authChanges:
provider: bedrock+bedrock.regionuv pip install -e '.[bedrock]'for boto3OPENAI_API_KEYin .envus.anthropic.claude-sonnet-4-6hermes -zdirectlyTested locally:
hermes doctorconfirms native Bedrock connectivity with IAM instance role.