Skip to content

fix(docs): pass missing sourceCode prop to Wrapper component#28901

Merged
keithwillcode merged 2 commits intomainfrom
devin/1776247454-fix-docs-build-sourceCode
Apr 15, 2026
Merged

fix(docs): pass missing sourceCode prop to Wrapper component#28901
keithwillcode merged 2 commits intomainfrom
devin/1776247454-fix-docs-build-sourceCode

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

What does this PR do?

Fixes the @calcom/docs build failure caused by a missing sourceCode prop on the Wrapper component in apps/docs/app/[[...mdxPath]]/page.tsx.

The Wrapper component (typed as MDXWrapper from Nextra) expects all properties from Omit<EvaluateResult, 'default'>, which includes toc, metadata, and sourceCode. The code was only destructuring and passing toc and metadata from importPage(), omitting sourceCode, which caused the TypeScript compilation error:

Property 'sourceCode' is missing in type '{ children: Element; toc: Heading[]; metadata: $NextraMetadata; }' but required in type 'Omit<EvaluateResult, "default">'.

The fix destructures sourceCode from the importPage() result and passes it to <Wrapper>.

Mandatory Tasks (DO NOT REMOVE)

  • I have self-reviewed the code (A decent size PR without self-review might be rejected).
  • I have updated the developer docs in /docs if this PR makes changes that would require a documentation change. N/A
  • I confirm automated tests are in place that prove my fix is effective or that my feature works. N/A — this is a one-line type fix; the build itself serves as verification.

How should this be tested?

Run the docs build:

cd apps/docs && yarn build

The build should succeed without the sourceCode type error.

Checklist

  • My code follows the style guidelines of this project
  • I have checked that my changes generate no new warnings

Link to Devin session: https://app.devin.ai/sessions/2931bc3673eb42cebc9d1bf2ba4bca02
Requested by: @keithwillcode

Co-Authored-By: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR that start with 'DevinAI' or '@devin'.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@vercel
Copy link
Copy Markdown

vercel bot commented Apr 15, 2026

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

Project Deployment Actions Updated (UTC)
docs Error Error Apr 15, 2026 1:11pm

Request Review

@hbjORbj hbjORbj enabled auto-merge (squash) April 15, 2026 13:12
@keithwillcode
Copy link
Copy Markdown
Contributor

This needs a force merge because Devin has been removed from this repo now and the Trust Check fails

@keithwillcode keithwillcode disabled auto-merge April 15, 2026 13:14
@keithwillcode keithwillcode merged commit a1587f2 into main Apr 15, 2026
44 of 47 checks passed
@keithwillcode keithwillcode deleted the devin/1776247454-fix-docs-build-sourceCode branch April 15, 2026 13:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants