Skip to content

Conversation

@jonathanlab
Copy link
Contributor

@jonathanlab jonathanlab commented Dec 1, 2025

Stores sessions in S3, also stores the claude session ID so we can restore context using their sdk

Copy link
Contributor Author

jonathanlab commented Dec 1, 2025

@wiz-7ad640923b
Copy link

wiz-7ad640923b bot commented Dec 1, 2025

Wiz Scan Summary

Scanner Findings
Vulnerability Finding Vulnerabilities 2 Medium 1 Low
Data Finding Sensitive Data -
Secret Finding Secrets -
IaC Misconfiguration IaC Misconfigurations -
SAST Finding SAST Findings -
Total 2 Medium 1 Low

View scan details in Wiz

To detect these findings earlier in the dev lifecycle, try using Wiz Code VS Code Extension.

@jonathanlab jonathanlab mentioned this pull request Dec 1, 2025
@jonathanlab jonathanlab marked this pull request as ready for review December 1, 2025 17:52
@jonathanlab jonathanlab requested a review from a team as a code owner December 1, 2025 17:52
const response = await fetch(logUrl);

// Handle S3 errors (e.g., file doesn't exist yet)
if (!response.ok) {
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we improve the error handling here?

@jonathanlab jonathanlab force-pushed the 12-01-feat_set_up_acp_and_claude_adapter branch from 9063af6 to b72851b Compare December 2, 2025 13:36
@jonathanlab jonathanlab force-pushed the 12-01-feat_add_session_loading branch from 2ba2644 to a4abecc Compare December 2, 2025 13:36
Copy link
Contributor Author

jonathanlab commented Dec 2, 2025

Merge activity

  • Dec 2, 4:14 PM UTC: A user started a stack merge that includes this pull request via Graphite.
  • Dec 2, 4:16 PM UTC: Graphite rebased this pull request as part of a merge.
  • Dec 2, 4:18 PM UTC: @jonathanlab merged this pull request with Graphite.

@jonathanlab jonathanlab changed the base branch from 12-01-feat_set_up_acp_and_claude_adapter to graphite-base/175 December 2, 2025 16:14
@jonathanlab jonathanlab changed the base branch from graphite-base/175 to main December 2, 2025 16:15
@jonathanlab jonathanlab force-pushed the 12-01-feat_add_session_loading branch from a4abecc to 8626a51 Compare December 2, 2025 16:15
@jonathanlab jonathanlab merged commit a44301a into main Dec 2, 2025
9 of 11 checks passed
@jonathanlab jonathanlab deleted the 12-01-feat_add_session_loading branch December 2, 2025 16:18
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.

3 participants