Skip to content

Add run artifacts CLI subcommand#724

Merged
ralyodio merged 1 commit into
profullstack:masterfrom
Autowebassat-blip:add-run-artifacts-command
Jun 14, 2026
Merged

Add run artifacts CLI subcommand#724
ralyodio merged 1 commit into
profullstack:masterfrom
Autowebassat-blip:add-run-artifacts-command

Conversation

@Autowebassat-blip

Copy link
Copy Markdown
Contributor

Refs #133.

Adds a sh1pt run artifacts <runId> subcommand alongside the existing run list, status, and logs commands. The command currently matches the stubbed run-command style and supports --json output for future API integration.

Changes:

  • register run artifacts <runId> with human-readable stub output
  • add --json output returning { id, artifacts: [] }
  • extend run command registration coverage to include artifacts

Tests:

  • git diff --check origin/master...HEAD
  • pnpm --filter @profullstack/sh1pt-cli test -- runs.test.ts blocked locally: pnpm not available in PATH

/claim #133

@greptile-apps

greptile-apps Bot commented Jun 13, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds sh1pt run artifacts <runId> as a stub subcommand alongside the existing list, status, and logs commands. The implementation follows the same pattern already established in the file, with --json emitting { id, artifacts: [] } and human-readable output using the dim kleur prefix.

  • runs.ts: New artifacts subcommand registered on runsCmd with --json support and a // TODO placeholder pointing at GET /v1/runs/:id/artifacts.
  • runs.test.ts: New test file asserting all four subcommand names (list, status, logs, artifacts) are present on runsCmd.commands.

Confidence Score: 5/5

The change is a self-contained stub command that mirrors three already-merged commands; no existing behaviour is altered.

The new artifacts subcommand is pure additive: it follows the exact same structure as status and logs, touches no shared state, and the only output paths are a console.log stub or a JSON.stringify of a hardcoded empty object. The accompanying test correctly validates command registration. Nothing in the changed code can regress existing functionality.

No files require special attention.

Important Files Changed

Filename Overview
packages/cli/src/commands/runs.ts Adds artifacts <runId> subcommand consistent with existing status and list stubs; --json path and human-readable path are both implemented.
packages/cli/src/commands/runs.test.ts New test file verifying all four subcommands (list, status, logs, artifacts) are registered on runsCmd.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[sh1pt run artifacts runId] --> B{--json flag?}
    B -- yes --> C["console.log(JSON.stringify({ id, artifacts: [] }))"]
    B -- no --> D["console.log(kleur.dim('[stub] run artifacts · id=...'))"] 
    C --> E[return]
    D --> F["// TODO: GET /v1/runs/:id/artifacts"]
Loading

Reviews (1): Last reviewed commit: "Add run artifacts command" | Re-trigger Greptile

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

15 similar comments
@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@github-actions

Copy link
Copy Markdown

🤖 Auto-rebase: The branch was rebased successfully locally but could not be pushed to the fork. Please enable 'Allow edits from maintainers' in the PR settings, or rebase manually: git fetch upstream master && git rebase upstream/master.

@ralyodio ralyodio merged commit fa173b9 into profullstack:master Jun 14, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants