Repository navigation
Refresh the edka-app skill from edkadigital/apps - #44
Merged
Merged
Conversation
The skill now says how to install the CLI when the shell has none, and how to name each image of an app that runs more than one. A release checks that this copy matches the one in edkadigital/apps.
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
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.
The release checks that
skills/edka-appmatches the skill in edkadigital/apps, and it no longer does: the skill there now says how to install the CLI when the shell has none, and how to name each image of an app that runs more than one. This ismake app-skill, so the next release can publish.go test ./internal/cli, which checks every command in the skill's code blocksNote
Low Risk
Documentation-only skill text; behavior change is limited to agent guidance, with CLI code blocks still covered by existing tests.
Overview
Syncs
skills/edka-app/SKILL.mdwith the upstream edkadigital/apps skill (viamake app-skill) so release checks pass.Before starting: The skill now treats CLI installation as a prerequisite, not only sign-in. If
edkais missing, it documents install via the curl script, Homebrew tap, or Windows PowerShell one-liner, then still directs users toedka loginwhenwhoamishows no organization—without asking for passwords or tokens.Multi-image packages: Under
template.yamlguidance, it adds rules for apps with more than one container image: the gateway-facing image stays on--image; every other image must be named from its repository across values keys, tag fields, labels, and workload/container names (with explicit examples and bans on generic or framework-based names likeworkerordjango).Reviewed by Cursor Bugbot for commit 73e8e9c. Configure here.