feat(code): upload cloud run attachments#1737
Merged
Conversation
Contributor
Author
This stack of pull requests is managed by Graphite. Learn more about stacking. |
This was referenced Apr 20, 2026
tatoalo
force-pushed
the
feat/agent-cloud-run-artifact-prompts
branch
from
22a3a1b to
001e087
Compare
tatoalo
force-pushed
the
feat/code-cloud-run-attachment-upload
branch
from
0643a0a to
1164474
Compare
tatoalo
force-pushed
the
feat/agent-cloud-run-artifact-prompts
branch
from
001e087 to
d7c168b
Compare
tatoalo
force-pushed
the
feat/code-cloud-run-attachment-upload
branch
from
1164474 to
3ea8477
Compare
tatoalo
force-pushed
the
feat/code-cloud-run-attachment-upload
branch
from
7e0cb60 to
da0cf6d
Compare
tatoalo
force-pushed
the
feat/agent-cloud-run-artifact-prompts
branch
from
d7c168b to
625f5ee
Compare
joshsny
approved these changes
Apr 21, 2026
| export const CLOUD_ATTACHMENT_MAX_SIZE_BYTES = 30 * 1024 * 1024; | ||
| export const CLOUD_PDF_ATTACHMENT_MAX_SIZE_BYTES = 10 * 1024 * 1024; | ||
|
|
||
| const CONTENT_TYPE_BY_EXTENSION: Record<string, string> = { |
Contributor
There was a problem hiding this comment.
do we need to maintain this locally, surely it's a solved problem somewhere?
| const ATTACHMENT_SOURCE = "posthog_code"; | ||
| const DEFAULT_CONTENT_TYPE = "application/octet-stream"; | ||
| export const CLOUD_ATTACHMENT_MAX_SIZE_BYTES = 30 * 1024 * 1024; | ||
| export const CLOUD_PDF_ATTACHMENT_MAX_SIZE_BYTES = 10 * 1024 * 1024; |
Contributor
There was a problem hiding this comment.
why do we do a separate size for pdfs?
tatoalo
force-pushed
the
feat/code-cloud-run-attachment-upload
branch
from
da0cf6d to
e6dbc09
Compare
tatoalo
force-pushed
the
feat/agent-cloud-run-artifact-prompts
branch
from
625f5ee to
7183c27
Compare
tatoalo
changed the base branch from
feat/agent-cloud-run-artifact-prompts
to
graphite-base/1737
tatoalo
force-pushed
the
graphite-base/1737
branch
from
7183c27 to
6a15adb
Compare
tatoalo
force-pushed
the
feat/code-cloud-run-attachment-upload
branch
from
e6dbc09 to
71f0037
Compare
tatoalo
force-pushed
the
feat/code-cloud-run-attachment-upload
branch
from
71f0037 to
4d4389a
Compare
Contributor
Author
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
2 participants
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.
Problem
The desktop app needs to upload task attachments to PostHog and send artifact references instead of embedding cloud prompt files inline.
cloud_agent_s3_backed_artifacts.mp4
Changes