Skip to content

feat: add show command to display collected analytics directly in the terminal - #3

Merged
sebastienthibaud merged 1 commit into
mainfrom
feat/add-analytics-show-command
Apr 30, 2026
Merged

sebastienthibaud merged 1 commit into
mainfrom
feat/add-analytics-show-command

Conversation

@sebastienthibaud

@sebastienthibaud sebastienthibaud commented Apr 27, 2026

Copy link
Copy Markdown
Contributor
image
image

Adds a new “show” command/CLI that reads collected analytics (timers + human estimates) from spec folders and renders a consolidated report directly in the terminal.

Changes:

  • Introduce src/show.ts to gather analytics from Markdown/JSON sources and render a terminal report (with optional project totals).
  • Wire the show command into the controller/extension surface area, plus add a ska-show CLI and a bash wrapper script.
  • Update docs and tests to cover the new show functionality; align Node built-in imports (node:*) and TS config.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds a new “show” command/CLI that reads collected analytics (timers + human estimates) from spec folders and renders a consolidated report directly in the terminal.

Changes:

  • Introduce src/show.ts to gather analytics from Markdown/JSON sources and render a terminal report (with optional project totals).
  • Wire the show command into the controller/extension surface area, plus add a ska-show CLI and a bash wrapper script.
  • Update docs and tests to cover the new show functionality; align Node built-in imports (node:*) and TS config.

Reviewed changes

Copilot reviewed 15 out of 17 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
tsconfig.json Adds Node types for TS compilation.
tests/run.ts Adds unit/integration tests for the new show command; updates imports to node:*.
src/timer.ts Switches Node built-in imports to node:*.
src/show.ts New analytics reader/renderer + CLI entrypoint.
src/purge.ts Switches Node built-in imports to node:*.
src/gitignore.ts Switches Node built-in imports to node:*.
src/estimator.ts Switches Node built-in imports to node:*.
src/controller.ts Registers a new show handler/command alongside purge.
src/config.ts Switches Node built-in imports to node:*.
scripts/bash/show.sh Adds a bash wrapper to render analytics via Node.
package.json Adds ska-show bin + setup/unlink scripts.
extension.yml Exposes a new speckit.analytics.show command.
doc/summary.md Links new show-command documentation.
doc/show-command.md Adds documentation for installing/using show.
commands/speckit.analytics.show.md Adds Spec Kit command wrapper calling the show script.
bin/show.js Adds the ska-show CLI entrypoint.
README.md Documents the show command and installation options.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/show.ts Outdated
Comment thread doc/show-command.md Outdated
Comment thread src/show.ts Outdated
Comment thread src/show.ts Outdated
@sebastienthibaud
sebastienthibaud force-pushed the feat/add-analytics-show-command branch from 38de868 to 62decfe Compare April 27, 2026 17:07
Comment thread package.json Outdated

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 14 out of 15 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread scripts/bash/show.sh Outdated
Comment thread doc/show-command.md
Comment thread README.md

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 14 out of 15 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/show.ts
Comment thread src/show.ts Outdated
Comment thread src/show.ts Outdated
Comment thread commands/speckit.analytics.show.md Outdated
@sebastienthibaud
sebastienthibaud force-pushed the feat/add-analytics-show-command branch from f61365d to 0dcea48 Compare April 30, 2026 08:39
@sebastienthibaud
sebastienthibaud merged commit 2aa6d28 into main Apr 30, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants