Skip to content

fix(loop): escape argument errors - #16

Merged
gold-silver-copper merged 5 commits into
mainfrom
fix/meta-stable-dotmeta
Oct 28, 2026
Merged

gold-silver-copper merged 5 commits into
mainfrom
fix/meta-stable-dotmeta

Conversation

@gold-silver-copper

@gold-silver-copper gold-silver-copper commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor
  • loop's argument errors are rendered from the escaped arguments, so a value can't move the cursor or forge a line (new meta_core dependency, for escape, declared as loop_lib does).
  • --verbose escapes the config path and command it echoes.
  • Stale comment fix ("Parse the .meta file": it's .looprc).
  • Invalid UTF-8 arguments get clap's own error (it doesn't echo the value).

Part of the meta stability set: gitkb/meta#128 has the full description and how it was verified. Merge together with it (same branch name, so CI builds them together).

Implements [[tasks/harmony-2793]]

@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 4c05f2ae-3c01-40e1-bbac-300c12653102
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

[[tasks/harmony-2793]]
@gold-silver-copper gold-silver-copper changed the title docs(loop): the config parsed is .looprc fix(loop): escape argument errors Oct 27, 2026
Its message doesn't echo the value, so the escaping fallback isn't needed.

Implements [[tasks/harmony-2793]]
@gold-silver-copper
gold-silver-copper merged commit d551fc8 into main Oct 28, 2026
7 checks passed
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