🤖 Support multiline continue messages in /compact #255
+196
−22
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Add rawInput to parser infrastructure to preserve newlines - Update /compact handler to parse multiline continue messages - Lines after first line become the continue message - -c flag still works for backwards compatibility (undocumented) - Add 9 new test cases for multiline parsing - Update docs to show multiline syntax (removed -c from examples) Benefits: - More ergonomic: no need for quotes or escaping - More readable: natural multiline format - Fully backwards compatible: -c flag still works Example: /compact -t 5000 Continue implementing the auth system. Make sure to add tests for edge cases.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
The previous commit broke multiline support for commands like /providers
by only parsing tokens from the first line. This commit fixes the issue:
- Parse tokens from full input (as before) to support all commands
- /compact handler checks for newlines in rawInput to distinguish
between single-line positional args (error) vs multiline content (valid)
- Add 4 new tests to ensure /providers, /model, /truncate work with newlines
The fix:
- Parser: parse tokens from full string (newlines treated as whitespace)
- /compact: detect multiline via rawInput.includes('\n'), skip positional
arg check if multiline (since multiline content appears as positional args)
This preserves backwards compatibility while enabling multiline /compact.
ammario
approved these changes
Oct 15, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Overview
Refactor
/compactcommand to support multiline continue messages. Everything after the first line becomes the continue message, making it much more ergonomic than the old-c "message"syntax.Old syntax (still works for backwards compat):
New syntax:
Changes
Parser Infrastructure
rawInputfield toSlashCommandHandlerArgsto pass raw input with newlines preservedCompact Command Handler
-cflag if present (backwards compatibility)Tests
-tflag-cflag takes precedence (backwards compat)Documentation
docs/context-management.mdwith multiline syntax examplesdocs/prompting-tips.mdto show multiline usage-cflag from documented syntax (but it still works)Benefits
-cflag still works (just undocumented)Testing
Files Changed
src/utils/slashCommands/types.ts- Add rawInput to handler argssrc/utils/slashCommands/parser.ts- Pass rawInput to handlerssrc/utils/slashCommands/registry.ts- Update compact handlersrc/utils/slashCommands/compact.test.ts- Add multiline testsdocs/context-management.md- Update syntax and examplesdocs/prompting-tips.md- Update examplesNet change: +132 lines / -21 lines = +111 lines
Generated with
cmux