Skip to content

Root package.json name and description update#13

Merged
curtisman merged 1 commit intomicrosoft:mainfrom
curtisman:description
Aug 12, 2024
Merged

Root package.json name and description update#13
curtisman merged 1 commit intomicrosoft:mainfrom
curtisman:description

Conversation

@curtisman
Copy link
Copy Markdown
Member

No description provided.

@curtisman curtisman enabled auto-merge August 12, 2024 17:16
@curtisman curtisman added this pull request to the merge queue Aug 12, 2024
Merged via the queue into microsoft:main with commit bbf4d53 Aug 12, 2024
curtisman added a commit to curtisman/TypeAgent that referenced this pull request Apr 4, 2026
- Fix mergeSepMode comment: 'at position' → 'at position - 1'
- Expand effectiveTrailingSep JSDoc with specific recomputation details
- Clarify 'wildcard-at-EOI string states' → 'states with a string next-part'
- Change computeRangeNeedsSep return type to tri-state (needsSep/noSep/drop)
- actionGrammar.md: 'Fixed:' → 'Handled via:' in summary table
- actionGrammar.md: plain-text three-way table → proper markdown table
- completion.md: add inline invariant descriptions (#3, microsoft#7, microsoft#8, microsoft#9, microsoft#13)
- grammarStore.ts: add cross-reference to filterSepConflicts
- Test: 'wins over optional' → 'is strongest merged mode'
curtisman added a commit to curtisman/TypeAgent that referenced this pull request Apr 8, 2026
Replace the merged separator-mode model with per-group separator modes.
Each CompletionGroup now carries its own separatorMode; the shell's
SepLevel model shows/hides groups based on trailing separator state
instead of the grammar layer filtering conflicting candidates.

- Refactor hasWhitespaceBefore() to inferSeparatorMode(), handling
  index=0 and returning SeparatorMode directly
- NFA completions use autoSpacePunctuation (consumer resolves per-item)
- Remove conflict-filtering docs; update invariant microsoft#13 and separator
  mode table
- Simplify toPartitions() — empty partitions are harmless
- Add debugCompletion channel to grammarStore
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