fix: DS indexing now runs with correct context

[liamdebeasi]

Previously the AsyncLocalStorage was not defined when making these requests. As a result, Builder credentials were not being resolved, and the request failed.

[github-actions]

Here's a visual recap of what changed:

Open the full interactive recap

[liamdebeasi]

@builderio-bot look at latest PR feedback and fix anything you agree with. Be skeptical. Reply to every comment (directly on the comment thread of each comment) if you fixed it or not and why

[builder-io-integration]

Both review comments addressed and replied to, changes committed and pushed to the PR branch.

Summary: Both flagged issues were real bugs and I fixed them:

• Design template (templates/design/server/handlers/index-design-system-with-builder.ts): was passing the stale sign-in-time session.orgId into the new request context and to the proxy write. Now resolves the live active org via getOrgContext (falling back to session.orgId if org tables aren't available).
• Slides template (templates/slides/server/handlers/index-design-system-with-builder.ts): was passing session.orgId as preResolvedContext, which bypassed resolveSlidesRequestAuthContext's live-org resolution entirely. Removed that argument so it resolves live.

Updated both handler tests accordingly, verified all tests in both templates pass (unrelated Playwright browser-launch failures in design are pre-existing environment issues, not caused by this change), replied to both PR review threads explaining the fix, and pushed the commit.

---

⠀
🟢 Fixed both org-resolution bugs flagged in PR review, tests pass, replies posted, and changes pushed.

---

  

[builder-io-integration]

Builder reviewed your changes — looks good ✅

Review Details

Code Review Summary

This incremental review covers the latest 120-line update to the Design and Slides Builder-backed design-system indexing handlers. The previously reported organization-scoping issues are fixed: Design now resolves the live organization with getOrgContext before establishing AsyncLocalStorage, and Slides now allows withSlidesRequestContext to perform its normal live-org resolution instead of supplying stale session context. Both the Builder indexing request and the proxy persistence remain inside the request context, preserving credential and tenant scoping.

The current changes are consistent with the existing handler behavior and error paths. Targeted tests reported by the review pass, and no new confirmed actionable bugs were found. One review agent raised concerns about pre-existing differences in Slides upload validation and the new test's mocking boundary; these are not regressions caused by the current functional change and were not submitted as findings.

Risk assessment: Standard risk (server-side request context, authentication, and organization scoping).

🧪 Browser testing: Skipped — PR only modifies backend/server handlers and tests, with no user-facing browser UI impact.