Skip to content

feat: show fields inline in view commands and remove field list#111

Merged
angeloashmore merged 1 commit intoaa/remote-modelingfrom
claude/reverent-elion
Apr 10, 2026
Merged

feat: show fields inline in view commands and remove field list#111
angeloashmore merged 1 commit intoaa/remote-modelingfrom
claude/reverent-elion

Conversation

@angeloashmore
Copy link
Copy Markdown
Member

@angeloashmore angeloashmore commented Apr 10, 2026

Resolves: #102

Description

type view and slice view now display a summary of fields per tab (types) or per variation (slices). Each field entry shows ID, type, label, and placeholder if set. field list has been removed since it's now redundant.

Checklist

  • A comprehensive Linear ticket, providing sufficient context and details to facilitate the review of the PR, is linked to the PR.
  • If my changes require tests, I added them.
  • If my changes affect backward compatibility, it has been discussed.
  • If my changes require an update to the CONTRIBUTING.md guide, I updated it.

Preview

How to QA 1

  1. node --run build
  2. npx prismic type view "<type name>" — fields should appear grouped by tab
  3. npx prismic slice view "<slice name>" — fields should appear grouped by variation
  4. npx prismic field list — should no longer be a valid command

Note

Medium Risk
Moderate risk because it removes the prismic field list CLI command (breaking change) and changes the human-readable output of slice view/type view, which may affect scripts or user workflows.

Overview
slice view and type view now print fields inline: output is expanded to list fields grouped by variation (slices) or tab (types), including field id, type, optional label, and optional placeholder.

Removes prismic field list by deleting the command and its tests, and updating CLI routing accordingly; tests for slice view/type view are updated/extended to assert the new output.

Reviewed by Cursor Bugbot for commit daaf923. Bugbot is set up for automated code reviews on this repo. Configure here.

Footnotes

  1. Please use these labels when submitting a review:
    ❓ #ask: Ask a question.
    💡 #idea: Suggest an idea.
    ⚠️ #issue: Strongly suggest a change.
    🎉 #nice: Share a compliment.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit daaf923. Configure here.

const label = (config?.label as string) || "";
const placeholder = config?.placeholder ? `"${config.placeholder}"` : "";
console.info(` ${[id, field.type, label, placeholder].filter(Boolean).join(" ")}`);
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Duplicated field display logic across two files

Low Severity

The field formatting logic (extracting config, label, placeholder, and joining them with filter(Boolean).join) is completely duplicated between slice-view.ts and type-view.ts. If the display format ever needs updating (e.g., adding a new field attribute or changing formatting), both files need identical changes, risking inconsistency.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit daaf923. Configure here.

@angeloashmore angeloashmore merged commit 75f5bc9 into aa/remote-modeling Apr 10, 2026
13 checks passed
@angeloashmore angeloashmore deleted the claude/reverent-elion branch April 10, 2026 21:57
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