refactor(cli): drop the artifacts command#2
Merged
Conversation
The `artifacts` command listed a single-entry "catalog" (SLO params), a
vestigial pre-v5 surface from when the language was meant to grow more
artifact types. Since v5.0.0 closed that direction ("Artifacts Be Gone"),
the command perpetuates a deprecated mental model — discovery of SLO
fields is already covered by the LSP, the tour, and compile errors.
Removes the CommandSpec, dispatcher arm, handler, the only-caller
`pretty_print_slo_params` display helper, and tests. Strips the four
references to `caffeine artifacts` from `explanations.gleam` error
prose; keeps surrounding "artifact" concept words intact since the
internal compiler ontology is unchanged.
Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.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
1 participant
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.
Summary
caffeine artifactsCLI command, a vestigial pre-v5 surface that listed a single-entry "catalog" (SLO params) — the architectural concept it implied was abandoned in v5.0.0 ("Artifacts Be Gone").caffeine artifactsfromexplanations.gleamerror prose (E301, E402, E500). Surrounding "artifact" / "standard-library artifact" / "artifact call" concept words are preserved — only the dead command pointer is gone.pretty_print_slo_params(and its test) fromdisplay.gleamsince the removed handler was its only caller.The discovery job this command performed — "what fields does an SLO take?" — is already covered by the LSP, the tour, and compile errors.
Adjacent commands (
compile,format,types,explain) are unchanged.Test plan
gleam buildclean incaffeine_cli/gleam test— 145 tests passcaffeine artifactsand confirm the unknown-command did-you-mean kicks in (expect a suggestion oftypessince it's the closest known command at edit distance 6 — likely no suggestion, just the known-commands list)caffeine --helpandcaffeine help <cmd>render withoutartifacts🤖 Generated with Claude Code