Conversation
Add a Cursor skill that teaches agents to build ldcli locally, point it at staging, choose validation that fits the change, test safely against shared staging, and record a human-style CLI video for every PR. Wire the skill and the video requirement into AGENTS.md so agents discover it. Co-authored-by: Ramon Niebla <nieblara@users.noreply.github.com>
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.
Requirements
ldclilocally (see demo below)Related issues
None.
Describe the solution you've provided
Equips agents to validate CLI changes the way a human would, and requires proof in every PR:
.cursor/skills/validate-ldcli-changes/SKILL.mdcovering:make build→ test the./ldcliyou just built).https://app.staging.launchdarkly.com) viaLD_BASE_URI/LD_ACCESS_TOKENfrom Cursor Secrets — never production, never committed tokens.dev-server, interactivesetup/login).AGENTS.mdnow has a "Validating CLI changes" section pointing at the skill and stating the staging-validation + PR-video requirement, so agents discover it automatically. (CLAUDE.mdsymlinks toAGENTS.md.)Describe alternatives you've considered
AGENTS.md: kept a short pointer there but moved the detailed workflow into a skill so it loads on demand for CLI-change tasks and stays maintainable.LD_ACCESS_TOKEN), never in the repo, config file, logs, or the video.Additional context
Demo of the documented workflow (this is also the style of human-driven CLI video the skill requires in every PR). It builds the binary, points
LD_BASE_URIat staging, and runs commands live.whoamiandflags listintentionally show the auth-required error — theflags listmessage derivinghttps://app.staging.launchdarkly.com/settings/authorizationconfirms the staging wiring. No secrets appear on screen.Note: authenticated staging validation (e.g.
whoami, real resource lifecycles) requires anLD_ACCESS_TOKENstaging secret, which is not present in this environment. The demo validates the local, no-auth portions of the documented workflow (build, config, output formatting, and the auth-required error path that confirms staging wiring). Add anLD_ACCESS_TOKENstaging secret to enable full authenticated staging runs.