feat: print all entries and schema URLs in non-TTY mode (#2226)

[knullptr]

In non-TTY environments (CI/CD, AI coding agents, piped output), the interactive loop was skipped and only the first entry's QR code was rendered — which is useless in these contexts.

Now registerConsoleShortcuts detects non-TTY and prints a structured list of all entries with all schema URLs, buffered into a single write to avoid interleaving with other async output.

Summary by CodeRabbit

• New Features
  • Non-TTY support: plugin now prints all entries and all schema URLs in non-interactive environments and invokes provided callbacks for each URL.
• Tests
  • Added test coverage for non-TTY behavior, ensuring output content, callback invocation, and no interactive loop.
• Chores
  • Published a patch changeset documenting the non-TTY output behavior.

Checklist

• Tests updated (or not required).
• Documentation updated (or not required).
• Changeset added, and when a BREAKING CHANGE occurs, it needs to be clearly marked (or not required).

[changeset-bot]

🦋 Changeset detected

Latest commit: 55273a4

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

This PR includes changesets to release 1 package

Name	Type
@lynx-js/qrcode-rsbuild-plugin	Patch

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

[coderabbitai]

📝 Walkthrough

Walkthrough

A non-TTY handling path was added to the QR code plugin's registerConsoleShortcuts: when stdin or stdout is not a TTY, a new printNonTTY function prints all entries and their schema URLs in a structured format, invokes onPrint callbacks, and returns early instead of running the interactive loop.

Changes

Cohort / File(s)	Summary
QR Code Plugin Implementation packages/rspeedy/plugin-qrcode/src/shortcuts.ts	Adds non-TTY branch that unconditionally calls loop but returns early for non-TTY by invoking new async printNonTTY(options) which prints all entries and schema URLs and triggers onPrint callbacks.
QR Code Plugin Tests packages/rspeedy/plugin-qrcode/test/shortcuts.test.ts	Adds non-TTY mode test suite: simulates non-TTY stdio, asserts single write containing all entries and Lynx bundle URLs, verifies onPrint called for every generated URL, ensures interactive prompts are not invoked, and tests multiple schema URLs per entry.
Release Notes .changeset/buffer-nontty-output.md	New changeset documenting patch release: in non-TTY environments print all entries with all schema URLs rather than only a single QR code.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related issues

• [Feature]: Display all entries and schema URLs in non-TTY terminals #2226: Implements non-TTY code path and printNonTTY behavior to print all entries and schema URLs as requested by the issue.

Possibly related PRs

• fix: allow rspeedy to print output chunk urls with qrcode plugin #1921: Related changes to QR code URL-printing behavior; both PRs adjust printing semantics in non-interactive contexts.
• fix(plugin-qrcode): skip interactive loop in non-TTY environments (#2201) #2202: Modifies non-TTY handling in the same shortcuts.ts code path; both guard against entering the interactive loop when stdin is not a TTY.

Suggested reviewers

• upupming
• colinaaa
• luhc228

Poem

🐰 A hop through code, no TTY to bind,

I print each link for every find.
No prompts to chase, just URLs in sight,
Each entry shown in plain daylight,
A rabbit cheers for output kind.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title 'feat: print all entries and schema URLs in non-TTY mode' clearly and accurately summarizes the main change - adding support for printing all entries and schema URLs in non-TTY environments, which directly matches the file changes and PR objectives.
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@packages/rspeedy/plugin-qrcode/src/shortcuts.ts`:
- Line 94: S_BAR_END is being pushed raw into the lines array while S_BAR
symbols are wrapped with color.gray; update the usage where
lines.push(`${S_BAR_END}`) is called so S_BAR_END is wrapped with color.gray
(same wrapper used for S_BAR) to keep bar symbol coloring consistent (locate the
lines.push call for S_BAR_END and mirror the color.gray(...) usage applied to
S_BAR).

🧹 Nitpick comments (1)

packages/rspeedy/plugin-qrcode/test/shortcuts.test.ts (1)

55-73: Consider using afterEach for writeSpy cleanup to guard against assertion failures.

If an assertion throws before writeSpy.mockRestore() is reached, the spy leaks into subsequent tests. Moving the restore into an afterEach (or using vitest's vi.restoreAllMocks config) would be more robust.

Also applies to: 108-131

[relativeci]

Web Explorer

#7772 Bundle Size — 383.74KiB (0%).

55273a4(current) vs 98b2504 main#7766(baseline)

Bundle metrics  no changes

	Current #7772	Baseline #7766
Initial JS	154.88KiB	154.88KiB
Initial CSS	35.06KiB	35.06KiB
Cache Invalidation	0%	0%
Chunks	8	8
Assets	8	8
Modules	238	238
Duplicate Modules	16	16
Duplicate Code	2.99%	2.99%
Packages	4	4
Duplicate Packages	0	0

Bundle size by type  no changes

	Current #7772	Baseline #7766
JS	252.83KiB	252.83KiB
Other	95.85KiB	95.85KiB
CSS	35.06KiB	35.06KiB

Bundle analysis report Branch knullptr:feat/no-tty Project dashboard

---

^{Generated by RelativeCI Documentation Report issue}

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

[codspeed-hq]

Merging this PR will degrade performance by 5.47%

⚡ 1 improved benchmark
❌ 1 regressed benchmark
✅ 61 untouched benchmarks
⏩ 3 skipped benchmarks¹

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

	Benchmark	BASE	HEAD	Efficiency
⚡	transform 1000 view elements	45.1 ms	40.4 ms	+11.46%
❌	basic-performance-small-css	7.6 ms	8 ms	-5.47%

---

_{Comparing knullptr:feat/no-tty (55273a4) with main (98b2504)}

Footnotes

1. 3 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩

[upupming]

LGTM, thanks!