fix(appstore): catalogue list uses DisplayName headline, not full Description (PILOT-405)#362
fix(appstore): catalogue list uses DisplayName headline, not full Description (PILOT-405)#362matthew-pilot wants to merge 2 commits into
Conversation
|
/openclaw merge |
9a8181f to
554ddda
Compare
…cription (PILOT-405) The catalogue list previously printed the full Description field for every entry, which is designed to be multi-sentence prose (especially in v2 catalogue entries where display_name is the short headline). This contradicts the intended UX: list = quick glance, view = full detail. Change: the text-mode listing now prints DisplayName when present, falling back to Description for v1 catalogue entries that lack the newer field. The --json output is unchanged — it still emits the entire catalogueEntry struct so downstream consumers are unaffected. Added explicit tests for: - DisplayName is used as headline when present (full Description NOT leaked) - Fallback to Description when DisplayName is absent (v1 back-compat) Closes PILOT-405
2f3a679 to
13ce42a
Compare
|
🛠️ Merge blocked by ruleset All checks are green, the PR is approved, and I have @TeoSlayer — this needs a manual merge or adding matthew-pilot to the ruleset bypass actors. Merge state
|
|
🔄 Merge sweep - 2026-07-17 22:08 UTC All checks still ✅ green (including govulncheck), APPROVED, and has @TeoSlayer — this needs a manual merge or adding matthew-pilot as a bypass actor on the ruleset. |
|
🛠️ Merge-and-cleanup check complete. PR #362 is mergeable, all checks green, and approved. However, auto-merge authorization is required per policy ( Ready for human action — this PR can be merged via the GitHub UI or by commenting |
|
🔄 Merge sweep - 2026-07-18 01:08 UTC Same status — merge blocked by protected branch ruleset ( To unblock:
|
2 participants
What
The
pilotctl appstore cataloguelist now prints the short headline (DisplayName) instead of the full Description for each app entry.Why
The catalogue list is a quick-glance surface — the full Description is meant for
pilotctl appstore view <id>. When a v2 catalogue entry has bothdisplay_name(short) anddescription(long prose), the list was incorrectly printing the long description, contradicting the intended UX.Changes
DisplayNamewhen present, fall back toDescriptionfor v1 back-compat--jsonoutput: unchanged (emits fullcatalogueEntrystruct)Verification
go build ./cmd/pilotctl/✓go vet ./cmd/pilotctl/✓go test -run 'TestCmdAppStoreCatalogue' ./cmd/pilotctl/✓ (all 6 tests pass)2 files changed, 54 insertions(+), 1 deletion(-)
Closes PILOT-405