fix: fall back to summary when title is empty in formatCell

[robzolkos]

Problem

basecamp reports schedule --md shows blank names for every schedule entry:

## 82 upcoming items (43 entries, 31 recurring, 8 assignables)

- **Schedule Entries:** , , , , , , , ...

Ref: https://3.basecamp.com/2914079/buckets/46292715/card_tables/cards/9717714333

Root cause

The bc3 reports/upcoming API uses the calendar partial (schedules/calendar/_entry.json.jbuilder) which emits summary but not title for schedule entries. The SDK deserialises the missing field to Go's zero value "".

The generic formatCell renderer, when displaying an array of maps (as schedule_entries is rendered inside the UpcomingScheduleResponse object), walks name → title → id to find a display label. Since title is present but empty, the type assertion succeeds with ok=true, an empty string is appended for each entry, and the join produces , , , , ....

Note: Schedule::Entry#title in bc3 delegates directly to summary, so the two fields are semantically identical — summary is just what the calendar API path exposes.

Fix

Add summary as a fallback between title and id in the map[string]any arm of formatCell. Also add empty-string guards on name and title so an empty field can't silently win over a non-empty one lower in the chain.

This is a generic renderer fix — any future resource that uses summary as its display name benefits automatically.

Testing

Extended TestFormatCellWithMapArray with three new cases:

• Schedule-entry shape (empty title, summary wins)
• Non-empty title still beats summary
• Empty name does not shadow a valid title

bin/ci passes.

---

Summary by cubic

Fixes blank schedule entry names in basecamp reports schedule --md. formatCell now falls back to summary when title is empty and skips empty name/title values.

• Bug Fixes
  • Added summary fallback between title and id in formatCell, and ignore empty name/title.
  • Expanded tests to cover schedule entries and label precedence.

^{Written for commit b78b5d1. Summary will update on new commits.}

[cubic-dev-ai]

No issues found across 2 files

[Copilot]

Pull request overview

Fixes blank labels when rendering schedule entries in basecamp reports schedule --md by improving the generic formatCell map-array label selection to ignore empty name/title and fall back to summary before id.

Changes:

• Update formatCell’s []any → map[string]any rendering to use name → title → summary → id, with empty-string guards.
• Add test coverage for summary fallback and empty-string guard behavior in map arrays.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
internal/output/render.go	Adjusts map-array label selection in formatCell to prevent empty title from producing blank rendered items and adds summary fallback.
internal/output/output_test.go	Extends TestFormatCellWithMapArray with cases covering summary fallback and empty-string guard behavior.

Tip

If you aren't ready for review, convert to a draft PR.
Click "Convert to draft" or run gh pr ready --undo.
Click "Ready for review" or run gh pr ready to reengage.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.