feat(query): replace readline with read-multiline in REPL by dqn · Pull Request #895 · tailor-platform/sdk

dqn · 2026-04-02T08:03:55Z

Replace Node.js readline with @toiroakr/read-multiline for native multiline editing in the query REPL

Before / After

Before: Single-line readline with manual line buffering. Multi-line SQL/GraphQL required pressing Enter after each line, with no way to edit previous lines.

After: True multiline editor with validation-based submit, undo/redo, and history navigation.

sql>
SELECT *
FROM users
WHERE id = 1;

Main Changes

Replace node:readline with @toiroakr/read-multiline for built-in multiline editing, validation, and history
Add persistent REPL history stored in XDG config directory (~/.config/tailor-platform/)
Simplify REPL loop by removing manual line buffering, abort controller wiring, and custom Ctrl+C handling
Update help text to reflect new key bindings (Ctrl+J for newline, undo/redo, history navigation)

Notes

History files are not yet scoped per workspace/profile (documented as TODO)
REPL commands and empty input are saved to history due to library limitations (documented as TODO)
In Claude Code, newlines are entered with Shift+Enter (macOS) or Ctrl+Enter (Windows), which may be confused with REPL submit

Replace node:readline/promises with @toiroakr/read-multiline for the interactive query REPL, enabling full multi-line editing with cursor navigation, undo/redo, bracketed paste, and persistent history. - Enter inserts newline, modified-Enter (Shift/Ctrl/Alt) submits - Validate on submit: SQL requires ';' terminator, GraphQL requires complete document - File-based history per engine (~/.config/tailor-platform/) - Remove obsolete line-buffering logic and interrupt handling

…text - Check raw value length instead of trimmed length on Ctrl+C cancel so whitespace-only drafts do not exit the REPL (regression from readline) - Replace hardcoded Enter/Shift+Enter key bindings in \help with terminal-agnostic Ctrl+J and a reference to the auto-detected footer

- Note that history is shared across workspaces/profiles - Note that empty input and REPL commands are saved to history

Use the discriminated union on QueryDispatchResult.engine to branch print logic after a single execute() call, removing the redundant per-engine assertion guards.

changeset-bot · 2026-04-02T08:04:03Z

🦋 Changeset detected

Latest commit: ba775d3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages

Name	Type
@tailor-platform/sdk	Minor
@tailor-platform/create-sdk	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

pkg-pr-new · 2026-04-02T08:04:55Z

Open in StackBlitz

npm i https://pkg.pr.new/@tailor-platform/create-sdk@895

commit: ba775d3

claude · 2026-04-02T08:06:13Z

📖 Docs Consistency Check

✅ No inconsistencies found between documentation and implementation.

Checked areas:

CLI command documentation (cli-reference.md and cli/*.md files)
CLI documentation generation config (docs.test.ts)
Project documentation (CLAUDE.md)
Example code patterns

Analysis:

The query command is intentionally undocumented:

Not included in docs.test.ts command list
No docs/cli/query.md file exists
Not referenced in any other documentation

All user-facing changes are properly reflected in the implementation:

REPL help text updated with new key bindings (Ctrl+J for newline, Ctrl+Z/Y for undo/redo, Up/Down for history)
Ctrl+C behavior documented (Cancel current input)
Persistent history feature is implementation detail (XDG config directory)

PR Changes Summary:

Replaced node:readline with @toiroakr/read-multiline for multiline editing
Added persistent REPL history (stored in ~/.config/tailor-platform/)
Updated REPL help text to reflect new key bindings
Removed resolveReplInterruptAction tests (function removed)

Since the query command has no external documentation and all user-visible information is displayed via the in-REPL help command (which was updated), there are no documentation files requiring changes.

devin-ai-integration

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 5 additional findings.

…multiline

toiroakr · 2026-04-03T12:55:13Z

packages/sdk/src/cli/query/index.ts

+  logger.log(
+    engine === "sql"
+      ? "Submit executes when input ends with ';'."
+      : "Submit executes when input is a complete GraphQL document.",
+  );


This might not be correct anymore.

Fixed in fd7b2336.

toiroakr

I would also like to support features such as syntax highlighting and automatic bracket insertion. Please refer to the read-multiline example.

Bumps @toiroakr/read-multiline to 0.3.0 which adds highlight and transform callback options used by the REPL syntax highlighting and auto-close bracket support. Adds sql-highlight for SQL tokenization and promotes graphql from a transitive to a direct dependency for GraphQL Lexer access.

Wire the new highlight and transform options from read-multiline 0.3.0 into the query REPL: per-engine syntax highlighting (SQL via sql-highlight, GraphQL via the official graphql Lexer) and auto-closing brackets with auto-indent on newline. The editor module is lazy-imported inside runRepl so graphql and sql-highlight are not pulled in on every CLI startup.

dqn · 2026-04-04T23:25:20Z

#895 (review)
Addressed in ba775d34.

github-actions · 2026-04-04T23:28:22Z

Code Metrics Report (packages/sdk)

	main (0be43dd)	#895 (a76d4f3)	+/-
Coverage	57.0%	57.5%	+0.5%
Code to Test Ratio	1:0.4	1:0.4	-0.1

Details

  |                    | main (0be43dd) | #895 (a76d4f3) |  +/-  |
  |--------------------|----------------|----------------|-------|
+ | Coverage           |          57.0% |          57.5% | +0.5% |
  |   Files            |            337 |            338 |    +1 |
  |   Lines            |          11035 |          11154 |  +119 |
+ |   Covered          |           6292 |           6418 |  +126 |
- | Code to Test Ratio |          1:0.4 |          1:0.4 |  -0.1 |
  |   Code             |          67220 |          67500 |  +280 |
+ |   Test             |          27489 |          27588 |   +99 |

Code coverage of files in pull request scope (46.8% → 63.5%)

Files	Coverage	+/-	Status
packages/sdk/src/cli/query/index.ts	49.4%	+2.6%	modified
packages/sdk/src/cli/query/repl-editor.ts	91.4%	+91.4%	added

SDK Configure Bundle Size

	main (0be43dd)	#895 (a76d4f3)	+/-
configure-index-size	12.69KB	12.69KB	0KB
dependency-chunks-size	33.36KB	33.36KB	0KB
total-bundle-size	46.04KB	46.04KB	0KB

Runtime Performance

	main (0be43dd)	#895 (a76d4f3)	+/-
Generate Median	2,683ms	2,603ms	-80ms
Generate Max	2,775ms	2,656ms	-119ms
Apply Build Median	2,702ms	2,625ms	-77ms
Apply Build Max	2,815ms	2,662ms	-153ms

Type Performance (instantiations)

	main (0be43dd)	#895 (a76d4f3)
tailordb-basic	42,484	42,484
tailordb-optional	3,826	3,826
tailordb-relation	3,970	3,970
tailordb-validate	2,824	2,824
tailordb-hooks	5,689	5,689
tailordb-object	11,470	11,470
tailordb-enum	2,720	2,720
resolver-basic	9,239	9,239
resolver-nested	25,626	25,626
resolver-array	17,862	17,862
executor-schedule	4,244	4,244
executor-webhook	883	883
executor-record	4,746	4,746
executor-resolver	4,273	4,273
executor-operation-function	877	877
executor-operation-gql	879	879
executor-operation-webhook	898	898
executor-operation-workflow	2,290	2,290

Reported by octocov

dqn added 9 commits April 1, 2026 17:52

docs: add TODO comments for REPL history improvement opportunities

256b2cf

- Note that history is shared across workspaces/profiles - Note that empty input and REPL commands are saved to history

docs: document history file race condition on fast re-prompt paths

66f59c1

refactor: consolidate SQL/GQL execute-and-print into single dispatch

8440825

Use the discriminated union on QueryDispatchResult.engine to branch print logic after a single execute() call, removing the redundant per-engine assertion guards.

refactor: simplify printReplHelp engine-specific message to ternary

59418c3

chore: revert unrelated .gitignore change

8b5cc49

fix: disable default prefix in read-multiline prompt

c87e719

chore: add changeset for query REPL multiline editor

a4b134d

dqn requested review from remiposo and toiroakr as code owners April 2, 2026 08:03

This comment has been minimized.

Sign in to view

devin-ai-integration bot reviewed Apr 2, 2026

View reviewed changes

Merge remote-tracking branch 'origin/main' into feat/query-repl-read-…

7bbd43e

…multiline

This comment has been minimized.

Sign in to view

Merge remote-tracking branch 'origin/main' into feat/query-repl-read-…

401da14

…multiline

This comment has been minimized.

Sign in to view

toiroakr reviewed Apr 3, 2026

View reviewed changes

reword REPL help text to describe validation-gated submit

fd7b233

toiroakr reviewed Apr 4, 2026

View reviewed changes

This comment has been minimized.

Sign in to view

dqn added 2 commits April 5, 2026 07:48

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(query): replace readline with read-multiline in REPL#895

feat(query): replace readline with read-multiline in REPL#895
dqn wants to merge 14 commits intomainfrom
feat/query-repl-read-multiline

dqn commented Apr 2, 2026 •

edited

Loading

Uh oh!

changeset-bot bot commented Apr 2, 2026 •

edited

Loading

Uh oh!

pkg-pr-new bot commented Apr 2, 2026 •

edited

Loading

Uh oh!

claude bot commented Apr 2, 2026 •

edited

Loading

Uh oh!

This comment has been minimized.

devin-ai-integration bot left a comment

Uh oh!

This comment has been minimized.

This comment has been minimized.

toiroakr Apr 3, 2026

Uh oh!

dqn Apr 4, 2026

Uh oh!

toiroakr left a comment

Uh oh!

This comment has been minimized.

dqn commented Apr 4, 2026 •

edited

Loading

Uh oh!

github-actions bot commented Apr 4, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

dqn commented Apr 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Before / After

Main Changes

Notes

Uh oh!

changeset-bot bot commented Apr 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

🦋 Changeset detected

Uh oh!

pkg-pr-new bot commented Apr 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

claude bot commented Apr 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

📖 Docs Consistency Check

Uh oh!

This comment has been minimized.

devin-ai-integration bot left a comment

Choose a reason for hiding this comment

✅ Devin Review: No Issues Found

Uh oh!

This comment has been minimized.

This comment has been minimized.

toiroakr Apr 3, 2026

Choose a reason for hiding this comment

Uh oh!

dqn Apr 4, 2026

Choose a reason for hiding this comment

Uh oh!

toiroakr left a comment

Choose a reason for hiding this comment

Uh oh!

This comment has been minimized.

dqn commented Apr 4, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

github-actions bot commented Apr 4, 2026

Code Metrics Report (packages/sdk)

Code coverage of files in pull request scope (46.8% → 63.5%)

SDK Configure Bundle Size

Runtime Performance

Type Performance (instantiations)

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

dqn commented Apr 2, 2026 •

edited

Loading

changeset-bot bot commented Apr 2, 2026 •

edited

Loading

pkg-pr-new bot commented Apr 2, 2026 •

edited

Loading

claude bot commented Apr 2, 2026 •

edited

Loading

dqn commented Apr 4, 2026 •

edited

Loading