Skip to content

Conversation

jog1t
Copy link
Contributor

@jog1t jog1t commented Sep 18, 2025

No description provided.

@jog1t jog1t mentioned this pull request Sep 18, 2025
Copy link

vercel bot commented Sep 18, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Preview Comments Updated (UTC)
rivet-cloud Error Error Sep 29, 2025 7:50pm
rivet-inspector Error Error Sep 29, 2025 7:50pm
rivet-site Error Error Sep 29, 2025 7:50pm
rivet-studio Error Error Sep 29, 2025 7:50pm

Copy link
Contributor Author

jog1t commented Sep 18, 2025

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more


How to use the Graphite Merge Queue

Add the label merge-queue to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

This stack of pull requests is managed by Graphite. Learn more about stacking.

@jog1t jog1t mentioned this pull request Sep 18, 2025
Copy link

claude bot commented Sep 18, 2025

Claude encountered an error —— View job


I'll analyze this and get back to you.

@jog1t jog1t marked this pull request as ready for review September 18, 2025 21:33
Copy link

pkg-pr-new bot commented Sep 18, 2025

Open in StackBlitz

npm i https://pkg.pr.new/rivet-dev/engine/@rivetkit/engine-runner@2945
npm i https://pkg.pr.new/rivet-dev/engine/@rivetkit/engine-runner-protocol@2945

commit: 8808a69

Copy link
Contributor

graphite-app bot commented Sep 18, 2025

Graphite Automations

"Test" took an action on this PR • (09/18/25)

1 assignee was added to this PR based on Kacper Wojciechowski's automation.

Copy link

claude bot commented Sep 22, 2025

Claude encountered an error —— View job


I'll analyze this and get back to you.

Copy link

claude bot commented Sep 22, 2025

Claude encountered an error —— View job


I'll analyze this and get back to you.

@jog1t jog1t force-pushed the 09-17-feat_cloud_billing branch from d6f2981 to 0494aa3 Compare September 26, 2025 21:55
@jog1t jog1t force-pushed the 09-18-feati_cloud_add_changelog branch from 64a73f8 to 4aaab18 Compare September 26, 2025 21:55
Copy link

claude bot commented Sep 26, 2025

Claude encountered an error —— View job


I'll analyze this and get back to you.

Copy link

claude bot commented Sep 26, 2025

Claude encountered an error —— View job


I'll analyze this and get back to you.

@jog1t jog1t force-pushed the 09-17-feat_cloud_billing branch from 0494aa3 to d0fa589 Compare September 29, 2025 19:27
@jog1t jog1t force-pushed the 09-18-feati_cloud_add_changelog branch from 4aaab18 to 8808a69 Compare September 29, 2025 19:27
Copy link

claude bot commented Sep 29, 2025

Claude encountered an error —— View job


I'll analyze this and get back to you.

Copy link

claude bot commented Sep 29, 2025

Claude encountered an error —— View job


I'll analyze this and get back to you.

Comment on lines +83 to +86
<a
className="flex gap-1.5 items-center flex-row-reverse text-right"
href={authors[0].socials.twitter}
>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The href attribute references authors[0].socials.twitter directly, but according to the schema definition in types.ts, this property is optional. This could lead to broken links if an author doesn't have a Twitter account. Consider adding a fallback:

href={authors[0].socials.twitter || '#'}

Or implement a more robust approach that selects the first available social link from the author's socials object, with a sensible default when none exist.

Spotted by Diamond

Fix in Graphite


Is this helpful? React 👍 or 👎 to let us know.

Copy link
Contributor

graphite-app bot commented Oct 1, 2025

Merge activity

  • Oct 1, 4:49 AM UTC: NathanFlurry added this pull request to the Graphite merge queue.
  • Oct 1, 4:50 AM UTC: CI is running for this pull request on a draft pull request (#3026) due to your merge queue CI optimization settings.
  • Oct 1, 4:51 AM UTC: Merged by the Graphite merge queue via draft PR: #3026.

graphite-app bot pushed a commit that referenced this pull request Oct 1, 2025
@graphite-app graphite-app bot closed this Oct 1, 2025
@graphite-app graphite-app bot deleted the 09-18-feati_cloud_add_changelog branch October 1, 2025 04:51
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