Skip to content

docs: note console.table quoting in skill list output - #47

Merged
ManningWorks merged 1 commit into
mainfrom
fix/issue-43-list-quoting-note
Oct 13, 2026
Merged

ManningWorks merged 1 commit into
mainfrom
fix/issue-43-list-quoting-note

Conversation

@ManningWorks

Copy link
Copy Markdown
Owner

Closes #43 (remaining half).

Issue #43 reported two things about the list output documented in
skills/projex/SKILL.md §3:

  1. The table has a leading (index) column not in the documented column
    list.
  2. Cell values are single-quoted in the real output, which the column
    description didn't mention.

Point 1 was already fixed on the skill branch in a181262 (PR #40, merged) —
the doc line now reads (index) | ID | Type | Status | Featured (★) | Name.
Point 2 was not. This PR adds the one-line note: the table is rendered via
Node's console.table, which auto-prepends the (index) column and
single-quotes string cell values, so parsers must strip the quotes.

Verified against live projex list output (2-row config, manual + npm
entries) — the rendered table matches the documented shape exactly.

@vercel

vercel Bot commented Oct 13, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
projex-docs Ready Ready Preview Oct 13, 2026 11:07pm UTC

@coderabbitai

coderabbitai Bot commented Oct 13, 2026

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

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

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: a312e6d6-b669-4664-908f-57e53be1b644
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

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

@ManningWorks
ManningWorks merged commit 46b6100 into main Oct 13, 2026
7 checks passed

This branch was successfully deployed

1 active deployment
Preview — bf16bb34 Deployed Oct 13, 2026 by vercel[bot]
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.

[skill] 'list' output table has a leading (index) column not in the skill's column description

1 participant