Repository navigation
Conversation
Uploading an image never exposed Trello's cover fields, so callers could not choose the picture on the front of a card. Optional setCover and cover fields on the existing tools write that cover without adding a tool.
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
get_cardalready returns it; nothing wrote it.setCoveron the existing attach tools sends Trello'ssetCoverquery parameter only when it is true.update_card_detailsacceptscoverAttachmentId(string to use an attachment already on the card,nullto clear the image),coverSize(normal|full), andcoverBrightness(light|dark). Those fields are added to the PUT body only when one of them is passed, so a name-only update does not sendcover.get_cardmarkdown names the attachment that is the cover. No new tool.Live check
On a disposable card, then archived:
setCover: trueon an uploaded image switched the cover to that attachment, and markdown named it. The card banner and the attachment's Cover label matched.coverAttachmentIdpluscoverSize: "full"moved the cover onto an earlier attachment. The board face used that image.coverAttachmentId: nullremoved the image cover and left both attachments in place.Trello still used the first uploaded image as the cover when
setCoverwas omitted and the card had no cover. The flag is not sent in that case. That is Trello's upload behavior, not a second write from this server.Test plan
bun run test:coverage(164 passed, 26 skipped; coverage watermark raised invitest.config.ts)bun run buildbun run test:coverageandbun run buildon the branch