Skip to content

feat: extend /info to accept optional time parameter#92

Merged
VarunGitGood merged 1 commit into
VarunGitGood:mainfrom
vikassaini77:issue-81-info-time-param
Jun 28, 2026
Merged

feat: extend /info to accept optional time parameter#92
VarunGitGood merged 1 commit into
VarunGitGood:mainfrom
vikassaini77:issue-81-info-time-param

Conversation

@vikassaini77

Copy link
Copy Markdown
Contributor

Closes #81

This PR extends the /info slash command to accept an optional time window parameter (e.g. 1h, 24h, yesterday).

Changes made:

  • Updated the /info command description in CommandPicker.tsx to mention the optional window arguments.
  • Modified ChatInput.tsx so submitting a command via Enter correctly preserves any typed arguments.
  • Updated the Turn type in types.ts to support an optional window parameter.
  • Updated page.tsx to parse the window argument from the command string and pass it down as a prop.
  • Updated ProjectOverview.tsx to receive the optional window prop and forward it to the api.projects.overview() API call.

Bare /info (no arg) keeps its current behaviour and anchors to the default 5-hour window.

@vercel

vercel Bot commented Jun 24, 2026

Copy link
Copy Markdown

@vikassaini77 is attempting to deploy a commit to the varun7singh's projects Team on Vercel.

A member of the Team first needs to authorize it.

@VarunGitGood VarunGitGood left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

LGTM. Both concerns verified clean:

  • Only info command exists, so ChatInput change is safe.
  • parse_window() falls back to 5h default on bad input — no injection risk.
  • 254/254 tests pass.

@VarunGitGood
VarunGitGood merged commit c61a698 into VarunGitGood:main Jun 28, 2026
2 of 3 checks passed
VarunGitGood added a commit that referenced this pull request Jul 5, 2026
feat: extend /info to accept optional time parameter
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.

Extend /info to accept an optional time parameter

2 participants