Skip to content

Conversation

@avivkeller
Copy link
Member

Copilot AI review requested due to automatic review settings December 25, 2025 21:09
@avivkeller avivkeller requested a review from a team as a code owner December 25, 2025 21:09
@vercel
Copy link

vercel bot commented Dec 25, 2025

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

Project Deployment Review Updated (UTC)
api-docs-tooling Ready Ready Preview Dec 25, 2025 9:10pm

@codecov
Copy link

codecov bot commented Dec 25, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.03%. Comparing base (a5258a9) to head (0d0c8b2).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #536   +/-   ##
=======================================
  Coverage   80.03%   80.03%           
=======================================
  Files         122      122           
  Lines       12066    12066           
  Branches      848      848           
=======================================
  Hits         9657     9657           
  Misses       2406     2406           
  Partials        3        3           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link

api-links Generator

apilinks.json
Expected values to be strictly deep-equal:
+ actual - expected
... Skipped lines

  {
    'Agent.defaultMaxSockets': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_agent.js#L227',
    'Buffer.alloc': 'https://github.com/nodejs/node/tree/HEAD/lib/buffer.js#L403',
    'Buffer.allocUnsafe': 'https://github.com/nodejs/node/tree/HEAD/lib/buffer.js#L417',
    'Buffer.allocUnsafeSlow': 'https://github.com/nodejs/node/tree/HEAD/lib/buffer.js#L429',
...
    'agent.addRequest': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_agent.js#L292',
+   'agent.createConnection': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_agent.js#L231',
-   'agent.createConnection': 'https://github.com/nodejs/node/tree/HEAD/lib/https.js#L326',
    'agent.createSocket': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_agent.js#L363',
    'agent.destroy': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_agent.js#L595',
+   'agent.getName': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_agent.js#L261',
-   'agent.getName': 'https://github.com/nodejs/node/tree/HEAD/lib/https.js#L484',
    'agent.keepSocketAlive': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_agent.js#L552',
    'agent.removeSocket': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_agent.js#L491',
    'agent.reuseSocket': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_agent.js#L588',
    'assert.assert': 'https://github.com/nodejs/node/tree/HEAD/lib/assert.js#L185',
    'asyncResource.asyncId': 'https://github.com/nodejs/node/tree/HEAD/lib/async_hooks.js#L229',
...
    'server.address': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L2221',
+   'server.close': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_server.js#L611',
+   'server.closeAllConnections': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_server.js#L621',
+   'server.closeIdleConnections': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_server.js#L633',
-   'server.close': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L2354',
-   'server.closeAllConnections': 'https://github.com/nodejs/node/tree/HEAD/lib/https.js#L120',
-   'server.closeIdleConnections': 'https://github.com/nodejs/node/tree/HEAD/lib/https.js#L122',
    'server.getConnections': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L2316',
    'server.listen': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L2038',
    'server.ref': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L2459',
+   'server.setTimeout': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_server.js#L649',
-   'server.setTimeout': 'https://github.com/nodejs/node/tree/HEAD/lib/https.js#L124',
    'server.unref': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L2468',
+   'server[SymbolAsyncDispose]': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_server.js#L617',
+   'server[undefined]': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_server.js#L656',
-   'server[SymbolAsyncDispose]': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L2394',
-   'server[undefined]': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L2423',
    'serverresponse._finish': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_server.js#L244',
    'serverresponse._implicitHeader': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_server.js#L351',
    'serverresponse.assignSocket': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_server.js#L294',
    'serverresponse.detachSocket': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_server.js#L305',
    'serverresponse.statusCode': 'https://github.com/nodejs/node/tree/HEAD/lib/_http_server.js#L267',
...
    'socket.addSourceSpecificMembership': 'https://github.com/nodejs/node/tree/HEAD/lib/dgram.js#L926',
+   'socket.address': 'https://github.com/nodejs/node/tree/HEAD/lib/dgram.js#L815',
-   'socket.address': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L655',
    'socket.bind': 'https://github.com/nodejs/node/tree/HEAD/lib/dgram.js#L267',
    'socket.close': 'https://github.com/nodejs/node/tree/HEAD/lib/dgram.js#L778',
+   'socket.connect': 'https://github.com/nodejs/node/tree/HEAD/lib/dgram.js#L412',
-   'socket.connect': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L1222',
    'socket.destroySoon': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L797',
    'socket.disconnect': 'https://github.com/nodejs/node/tree/HEAD/lib/dgram.js#L490',
    'socket.dropMembership': 'https://github.com/nodejs/node/tree/HEAD/lib/dgram.js#L911',
    'socket.dropSourceSpecificMembership': 'https://github.com/nodejs/node/tree/HEAD/lib/dgram.js#L944',
    'socket.end': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L736',
...
    'socket.read': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L780',
+   'socket.ref': 'https://github.com/nodejs/node/tree/HEAD/lib/dgram.js#L1001',
-   'socket.ref': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L1562',
    'socket.remoteAddress': 'https://github.com/nodejs/node/tree/HEAD/lib/dgram.js#L827',
    'socket.resetAndDestroy': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L742',
    'socket.resume': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L771',
    'socket.send': 'https://github.com/nodejs/node/tree/HEAD/lib/dgram.js#L615',
    'socket.sendto': 'https://github.com/nodejs/node/tree/HEAD/lib/dgram.js#L504',
...
    'socket.setTimeout': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L587',
+   'socket.unref': 'https://github.com/nodejs/node/tree/HEAD/lib/dgram.js#L1011',
-   'socket.unref': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L1576',
    'socket[SymbolAsyncDispose]': 'https://github.com/nodejs/node/tree/HEAD/lib/dgram.js#L802',
    'socket[kAfterAsyncWrite]': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L931',
    'socket[kReinitializeHandle]': 'https://github.com/nodejs/node/tree/HEAD/lib/net.js#L1290',
    'string_decoder.StringDecoder': 'https://github.com/nodejs/node/tree/HEAD/lib/string_decoder.js#L60',
    'timers.clearImmediate': 'https://github.com/nodejs/node/tree/HEAD/lib/timers.js#L226',

orama-db Generator

File Base Head Diff
orama-db.json 7.99 MB 7.99 MB +1.89 KB (+0.02%)

web Generator

File Base Head Diff
styles.css 121.55 KB 121.63 KB +83.00 B (+0.07%)

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR addresses a UI issue by adding a maximum width constraint to modal content links and updates the Orama search packages to their latest patch versions.

  • Adds max-width: 42rem to modal content links for better text overflow handling
  • Updates @orama/orama from 3.1.16 to 3.1.18
  • Updates @orama/ui from 1.5.3 to 1.5.4

Reviewed changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
src/generators/web/ui/index.css Adds max-width constraint to modal content anchor elements to prevent overflow
package.json Updates Orama package dependencies to latest patch versions
npm-shrinkwrap.json Updates lockfile with new package versions and reorganized dependency tree
Files not reviewed (1)
  • npm-shrinkwrap.json: Language not supported

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@ovflowd ovflowd merged commit f14b31c into main Dec 25, 2025
26 checks passed
@ovflowd ovflowd deleted the max-orama-width branch December 25, 2025 22:38
@ovflowd ovflowd added the fast track This PR can land before the typical review time, with a :+1: from collaborators label Dec 25, 2025
@ovflowd
Copy link
Member

ovflowd commented Dec 25, 2025

Fast tracking as a bug fix.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast track This PR can land before the typical review time, with a :+1: from collaborators

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants