Skip to content

✨ Load external components with xmd repl --include - #972

Merged
taras merged 1 commit into
mainfrom
agent/repl-include-968-codex-20261020
Oct 21, 2026
Merged

taras merged 1 commit into
mainfrom
agent/repl-include-968-codex-20261020

Conversation

@taras

@taras taras commented Oct 21, 2026

Copy link
Copy Markdown
Owner

Why

Authors can load their own components into an interactive session with deno task xmd repl --include ./components, addressing #968.

What changes

Before: the REPL has no --include option, so a component directory used by ordinary document runs cannot be selected for the interactive session.

After: repeated --include arguments select component directories in the authored order. The selected vocabulary is shared by execution, preflight and Sidekick documentation. Relative paths resolve from the invocation directory; omitted options preserve the existing bundled components.

How to verify it

Focused grammar/help, profile/execution and source terminal regressions pass. The terminal journey covers ordering, reverse order, omitted options, protected names and reopening completed history without repeating effects. Both original reviewers passed signed commit a4907b468777e4112f87ff80d26f0e692fa73e35.

Required CI and runner-produced test weights are pending. This remains a draft until delivery verification and final metadata review finish.

Scope

CLI argument handling, the existing REPL profile, focused regression coverage and the REPL specification.

Scope confirmation

  • Every changed file supports loading external components in the REPL.
  • Unrelated cleanup and formatting changes are excluded.
  • No generated files or new dependencies are included.
  • Required CI, measured weights and final editorial verification are complete.

@github-actions

Copy link
Copy Markdown

PR #972: ✨ Load external components with xmd repl --include

8 files, +687 / -39

Scope

🟡 726 lines changed. PRs under 400 receive more thorough review.

Structural

Oxlint structural signals:

  • no-unused-vars ×3: packages/cli/src/cli.ts
  • no-empty-function ×2: packages/cli/src/repl-profile.ts, packages/cli/src/cli.ts

Slop

Oxlint slop signals:

  • no-console ×3: packages/cli/src/cli.ts

Static Analysis

Oxlint: 16 diagnostics across 2 files (9 rules)
Density: 0.023 violations/added-line

no-unused-vars (3): packages/cli/src/cli.ts
no-console (3): packages/cli/src/cli.ts
no-empty-function (2): packages/cli/src/repl-profile.ts, packages/cli/src/cli.ts
unbound-method (2): packages/cli/src/cli.ts
consistent-return (2): packages/cli/src/cli.ts
no-unmodified-loop-condition (1): packages/cli/src/cli.ts
no-useless-spread (1): packages/cli/src/cli.ts
no-unsafe-type-assertion (1): packages/cli/src/cli.ts
no-floating-promises (1): packages/cli/src/cli.ts

Correctness

No extraneous code patterns detected.

@taras
taras marked this pull request as ready for review October 21, 2026 07:33
@taras
taras merged commit 3a1c04b into main Oct 21, 2026
45 checks passed
@taras
taras deleted the agent/repl-include-968-codex-20261020 branch October 21, 2026 07:33
@taras taras mentioned this pull request Oct 21, 2026
4 tasks done
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