Skip to content

Add streaming and JSON support to cli prompt command#6

Merged
curtisman merged 1 commit intomicrosoft:mainfrom
curtisman:prompt
Aug 9, 2024
Merged

Add streaming and JSON support to cli prompt command#6
curtisman merged 1 commit intomicrosoft:mainfrom
curtisman:prompt

Conversation

@curtisman
Copy link
Copy Markdown
Member

@curtisman curtisman commented Aug 9, 2024

  • Add flag --stream to enable streaming.
  • Add flag --json to enable JSON format.
  • Add flag --model to select model.
  • Add timing information to cli prompt command
  • Fix executable permissions for CLI entrypoint scripts.
  • Make sure streaming mode retain the "json_object" response format type flags.

@curtisman curtisman enabled auto-merge August 9, 2024 22:12
@curtisman curtisman added this pull request to the merge queue Aug 9, 2024
Merged via the queue into microsoft:main with commit fc4e476 Aug 9, 2024
curtisman added a commit to curtisman/TypeAgent that referenced this pull request Mar 28, 2026
- Group trailingSepAdvanced with the directionSensitive decision tree
  under a shared section header for better locality (microsoft#7)
- Add invariant comment: openWildcard requires a preceding keyword
  match so P > 0 whenever openWildcard is true (#1)
- Add JSDoc to couldBackUp return field in tryPartialStringMatch (microsoft#5)
- Compact equivalence analysis tables in actionGrammar.md from 5-column
  wide format to 3-column for readability in narrow contexts (microsoft#6)
curtisman added a commit to curtisman/TypeAgent that referenced this pull request Mar 29, 2026
Fix four bugs in grammarCompletion.ts exposed by a new truncated-forward
invariant (result on full input must equal forward on input[0..mpl]):

- Category 3a backward: fall through to collectPropertyCandidate when
  tryCollectBackwardCandidate returns false, so property slots are not
  silently dropped.

- Phase B1 backward separator-only gap: preserve fixedCandidates when
  advancing maxPrefixLength across a separator-only gap instead of
  clearing them.

- Range candidate truncation: pass prefix.substring(0, maxPrefixLength)
  to tryPartialStringMatch so range candidates don't peek beyond mpl.

- gapIsSeparatorOnly: use anchor !== maxPrefixLength instead of
  a partial keyword is present.

Add assertTruncatedForwardInvariant to testUtils.ts and explicit backward
test cases for each fix.
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.

1 participant