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 4 commits into
Conversation
|
/openclaw merge |
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:
|
|
🟢 All checks green, approved, mergeable. matthew-pilot cannot auto-merge due to branch protection ruleset restrictions. A human with bypass permission needs to merge this PR. Merge details:
|
|
🔄 Merge sweep - 2026-07-18 09:57 UTC Still blocked by the protected branch ruleset. All checks green, approved, no change in status. This PR has been ready for |
|
🧵 merge-and-cleanup scan complete (10:28 UTC) PR #362 ( ⏩ Ping: @TeoSlayer — can you merge on your end? 👆 Other 23 open PRs: no other matthew-pilot PRs eligible. Dependabot bumps and teammate PRs all need reviews before merging. |
|
🔄 Merge sweep — 2026-07-18 12:13 UTC PR #362 is still blocked by the protected branch ruleset — matthew-pilot does not have bypass privileges. All checks are green, approved by @TeoSlayer, and the code has been ready since July 8. @TeoSlayer — please merge from the GitHub UI when you get a chance. This PR has been pending for 10 days. |
|
Re-triggering CI checks — Snyk stuck in pending |
|
🔄 Merge sweep - 2026-07-18 14:16 UTC Still blocked by the
@TeoSlayer — please either:
|
…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
aeef7e6 to
949ae2d
Compare
|
🔄 Merge sweep — 2026-07-18 18:14 UTC Same story, still blocked by the What's needed: A human with bypass permission (you!) merges via the GitHub UI — it's a 1-click squash. This PR has been ready since July 8 (10+ days). |
|
🔄 Merge sweep — 2026-07-18 18:29 UTC Same story — still blocked by the All checks ✅ green, approved ✅ by TeoSlayer, has Action needed: Merge via GitHub UI (one-click squash) or add |
|
🔄 Merge sweep — 2026-07-18 18:59 UTC Still blocked by the Protected Branches ruleset (no bypass for bot accounts). All checks remain green ✅, still approved by @TeoSlayer ✅, has No other eligible PRs found across the org for automated merge. Action needed: Merge via GitHub UI (one-click squash) or add |
|
🔄 Merge sweep — 2026-07-18 20:14 UTC Same status — still blocked by the
No other eligible PRs found across all tracked repos. This PR has been ready since July 8 (over 10 days). A human with bypass permission must merge via the GitHub UI (one-click squash), or add |
|
🔄 Merge sweep — 2026-07-18 22:00 UTC Still blocked by the
No other eligible PRs found across all tracked repos. This is the only PR that meets all criteria (green checks + approval + merge authorization). This PR has been ready since July 8 (10 days). A human with bypass permission must merge via the GitHub UI (one-click squash), or add |
|
🔄 Merge sweep — 2026-07-19 09:53 UTC Still blocked by the
No other eligible PRs found across all tracked repos. This PR has been ready since July 8 (11 days). A human with bypass permission must merge via the GitHub UI (one-click squash), or add |
|
🔄 Merge sweep — 2026-07-19 11:39 UTC Still blocked by the
No other eligible PRs found across all tracked repos. This PR has been ready since July 8 (11 days). A human with bypass permission must merge via the GitHub UI (one-click squash), or add |
|
🔄 Merge sweep — 2026-07-19 13:11 UTC Still blocked by the
No other eligible PRs found across all tracked repos. All other open PRs either lack approval, have failing checks, or have merge conflicts. This PR has been ready since July 8 (11 days). A human with bypass permission must merge via the GitHub UI (one-click squash), or add |
|
🔄 Merge sweep — 2026-07-19 15:32 UTC Still blocked by the
No other eligible PRs found across all tracked repos. All other open PRs either lack approval, have failing checks, or have merge conflicts. This PR has been ready since July 8 (11 days). A human with bypass permission must merge via the GitHub UI (one-click squash), or add |
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