Skip to content

Pass listing summary to PR body#4195

Merged
lucaslyl merged 1 commit intomainfrom
CS-10433/listing-description-return-null-after-a-pr-open
Mar 17, 2026
Merged

Pass listing summary to PR body#4195
lucaslyl merged 1 commit intomainfrom
CS-10433/listing-description-return-null-after-a-pr-open

Conversation

@lucaslyl
Copy link
Contributor

linear: https://linear.app/cardstack/issue/CS-10433/listing-description-return-null-after-a-pr-open

The PR description showed "No description provided" because the listingSummary field was never included in the bot trigger event input. Now we extract listing.summary from the card and pass it as listingSummary so the bot-runner can display it in the PR body as a standalone block separated by dividers, rather than inline in a bullet.

Demo:

Before:
image

After:
image

@lucaslyl lucaslyl requested a review from a team March 17, 2026 03:46
@lucaslyl lucaslyl self-assigned this Mar 17, 2026
@github-actions
Copy link

Preview deployments

@github-actions
Copy link

github-actions bot commented Mar 17, 2026

Host Test Results

    1 files  ±0      1 suites  ±0   2h 31m 16s ⏱️ + 3m 22s
2 022 tests ±0  2 007 ✅ ±0  15 💤 ±0  0 ❌ ±0 
2 037 runs  ±0  2 022 ✅ ±0  15 💤 ±0  0 ❌ ±0 

Results for commit 314f847. ± Comparison against base commit dc8558c.

♻️ This comment has been updated with latest results.

@lucaslyl lucaslyl merged commit 2f9f5ee into main Mar 17, 2026
202 of 210 checks passed
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.

2 participants