Feat/make it easier to display table names #876

prakha · 2025-03-12T20:30:06Z

Issue

resolve:

Why is this change needed?

What would you like reviewers to focus on?

Testing Verification

What was done

🤖 Generated by PR Agent at `43ad3fc`

Enhanced the TableNode component by adding tooltip functionality to display table names and refactored imports for consistency.
Simplified the TableHeader component by removing tooltip functionality and directly displaying table names.
Added type definitions for Next.js cache configurations, including support for custom caching profiles.
Added type definitions for CSS module classes in the CookieConsent component.
Introduced multiple vendor chunk files for OpenTelemetry instrumentation, including Fastify, Redis, and API logs, as well as other dependencies like module-details-from-path and has-flag.
Added a middleware build manifest configuration for defining polyfill files and build-related settings.

Detailed Changes

Relevant files

Documentation

2 files

cache-life.d.ts `Added type definitions for Next.js cache configurations.` frontend/apps/erd-web/.next/types/cache-life.d.ts Added type definitions for Next.js cache configurations. Introduced `unstable_cacheLife` function with predefined profiles like "default", "seconds", "minutes", etc. Included support for custom caching profiles with configurable `stale`, `revalidate`, and `expire` properties.	+141/-0
CookieConsent.module.css.d.ts `Added type definitions for CookieConsent CSS module.` frontend/apps/erd-web/components/CookieConsent/CookieConsent.module.css.d.ts Added type definitions for CSS module classes in the CookieConsent component. Declared class names like `button`, `content`, `controls`, etc.	+11/-0

Enhancement

2 files

TableNode.tsx `Enhanced TableNode component with tooltip for table names.` frontend/packages/erd-core/src/features/erd/components/ERDContent/components/TableNode/TableNode.tsx Added tooltip functionality to display table names using `TooltipProvider`. Integrated `TooltipContent` to show table name on hover. Refactored imports for consistency with double quotes.	+48/-29
TableHeader.tsx `Simplified TableHeader component by removing tooltip.` frontend/packages/erd-core/src/features/erd/components/ERDContent/components/TableNode/TableHeader/TableHeader.tsx Removed tooltip functionality from `TableHeader`. Simplified the component by directly displaying the table name without a tooltip. Refactored imports for consistency with double quotes.	+18/-29

Dependencies

6 files

@opentelemetry+instrumentation-mongoose@0.46.0_@opentelemetry+api@1.9.0.js `Added vendor chunk for OpenTelemetry Mongoose instrumentation.` frontend/apps/erd-web/.next/server/vendor-chunks/@opentelemetry+instrumentation-mongoose@0.46.0_@opentelemetry +api@1.9.0.js Added vendor chunk file for OpenTelemetry Mongoose instrumentation. Includes source mapping and module exports for OpenTelemetry integration.	+65/-0
module-details-from-path@1.0.3.js `Added vendor chunk for module-details-from-path package.` frontend/apps/erd-web/.next/server/vendor-chunks/module-details-from-path@1.0.3.js Added vendor chunk file for `module-details-from-path` package. Provides functionality to extract module details from file paths.	+25/-0
@opentelemetry+instrumentation-fastify@0.44.1_@opentelemetry+api@1.9.0.js `Add OpenTelemetry Fastify instrumentation vendor chunk.` frontend/apps/erd-web/.next/server/vendor-chunks/@opentelemetry+instrumentation-fastify@0.44.1_@opentelemetry +api@1.9.0.js Added a new vendor chunk file for `@opentelemetry/instrumentation-fastify`. Includes multiple modules and utilities for OpenTelemetry instrumentation. Implements functionality for tracing and span management in Fastify. Contains autogenerated and source-mapped content for debugging.	+85/-0
has-flag@4.0.0.js Add `has-flag` utility vendor chunk. frontend/apps/erd-web/.next/server/vendor-chunks/has-flag@4.0.0.js Added a new vendor chunk file for `has-flag`. Provides a utility function to check for flags in command-line arguments. Includes source-mapped content for debugging.	+25/-0
@opentelemetry+api-logs@0.57.1.js `Add OpenTelemetry API logs vendor chunk.` frontend/apps/erd-web/.next/server/vendor-chunks/@opentelemetry+api-logs@0.57.1.js Added a new vendor chunk for OpenTelemetry API logs. Includes various classes and methods for logging functionality, such as `NoopLogger`, `ProxyLogger`, and `LogsAPI`. Provides utilities for managing global logger providers and compatibility checks.	+105/-0
@opentelemetry+redis-common@0.36.2.js `Add OpenTelemetry Redis common vendor chunk.` frontend/apps/erd-web/.next/server/vendor-chunks/@opentelemetry+redis-common@0.36.2.js Added a new vendor chunk for OpenTelemetry Redis common utilities. Introduced `defaultDbStatementSerializer` for serializing Redis commands based on predefined subsets. Includes regex-based logic for command argument serialization.	+25/-0

Configuration changes

1 files

middleware-build-manifest.js `Add middleware build manifest configuration.` frontend/apps/erd-web/.next/server/middleware-build-manifest.js Introduced a build manifest for middleware. Defined polyfill files, dev files, and other build-related configurations. Dynamically includes low-priority files based on the build ID.	+19/-0

Additional files

91 files

_events.json	+1/-0
app-build-manifest.json	+3/-0
build-manifest.json	+17/-0
.rscinfo	+1/-0
index.pack.gz.old	[link]
index.pack.gz.old	[link]
package-lock.json	+668/-0
package.json	+1/-0
react-loadable-manifest.json	+1/-0
_instrument_sentry_server_config_ts.js	+25/-0
app-paths-manifest.json	+1/-0
edge-instrumentation.js	+1943/-0
edge-runtime-webpack.js	+1146/-0
instrumentation.js	+438/-0
interception-route-rewrite-manifest.js	+1/-0
middleware-manifest.json	+6/-0
middleware-react-loadable-manifest.js	+1/-0
next-font-manifest.js	+1/-0
next-font-manifest.json	+1/-0
pages-manifest.json	+1/-0
server-reference-manifest.js	+1/-0
server-reference-manifest.json	+5/-0
633457081244afec._.hot-update.json	+1/-0
@opentelemetry+api-logs@0.53.0.js	+85/-0
@opentelemetry+api-logs@0.57.2.js	+105/-0
@opentelemetry+api@1.9.0.js	+465/-0
@opentelemetry+context-async-hooks@1.30.1_@opentelemetry+api@1.9.0.js	+55/-0
@opentelemetry+core@1.30.1_@opentelemetry+api@1.9.0.js	+405/-0
@opentelemetry+instrumentation-amqplib@0.46.1_@opentelemetry+api@1.9.0.js	+65/-0
@opentelemetry+instrumentation-connect@0.43.0_@opentelemetry+api@1.9.0.js	+75/-0
@opentelemetry+instrumentation-dataloader@0.16.0_@opentelemetry+api@1.9.0.js	+55/-0
@opentelemetry+instrumentation-express@0.47.0_@opentelemetry+api@1.9.0.js	+95/-0
@opentelemetry+instrumentation-fs@0.19.0_@opentelemetry+api@1.9.0.js	+75/-0
@opentelemetry+instrumentation-generic-pool@0.43.0_@opentelemetry+api@1.9.0.js	+45/-0
@opentelemetry+instrumentation-graphql@0.47.0_@opentelemetry+api@1.9.0.js	+105/-0
@opentelemetry+instrumentation-hapi@0.45.1_@opentelemetry+api@1.9.0.js	+75/-0
@opentelemetry+instrumentation-http@0.57.1_@opentelemetry+api@1.9.0.js	+65/-0
@opentelemetry+instrumentation-ioredis@0.47.0_@opentelemetry+api@1.9.0.js	+65/-0
@opentelemetry+instrumentation-kafkajs@0.7.0_@opentelemetry+api@1.9.0.js	+65/-0
@opentelemetry+instrumentation-knex@0.44.0_@opentelemetry+api@1.9.0.js	+75/-0
@opentelemetry+instrumentation-koa@0.47.0_@opentelemetry+api@1.9.0.js	+85/-0
@opentelemetry+instrumentation-lru-memoizer@0.44.0_@opentelemetry+api@1.9.0.js	+45/-0
@opentelemetry+instrumentation-mongodb@0.51.0_@opentelemetry+api@1.9.0.js	+65/-0
@opentelemetry+instrumentation-mysql2@0.45.0_@opentelemetry+api@1.9.0.js	+65/-0
@opentelemetry+instrumentation-mysql@0.45.0_@opentelemetry+api@1.9.0.js	[link]
@opentelemetry+instrumentation-nestjs-core@0.44.0_@opentelemetry+api@1.9.0.js	[link]
@opentelemetry+instrumentation-pg@0.50.0_@opentelemetry+api@1.9.0.js	[link]
@opentelemetry+instrumentation-redis-4@0.46.0_@opentelemetry+api@1.9.0.js	[link]
@opentelemetry+instrumentation-tedious@0.18.0_@opentelemetry+api@1.9.0.js	+65/-0
@opentelemetry+instrumentation-undici@0.10.0_@opentelemetry+api@1.9.0.js	+65/-0
@opentelemetry+instrumentation@0.53.0_@opentelemetry+api@1.9.0.js	+115/-0
@opentelemetry+instrumentation@0.57.1_@opentelemetry+api@1.9.0.js	+115/-0
@opentelemetry+instrumentation@0.57.2_@opentelemetry+api@1.9.0.js	+115/-0
@opentelemetry+resources@1.30.1_@opentelemetry+api@1.9.0.js	+255/-0
@opentelemetry+sdk-trace-base@1.30.1_@opentelemetry+api@1.9.0.js	+215/-0
@opentelemetry+semantic-conventions@1.27.0.js	+125/-0
@opentelemetry+semantic-conventions@1.28.0.js	+95/-0
@opentelemetry+semantic-conventions@1.30.0.js	+95/-0
@opentelemetry+sql-common@0.40.1_@opentelemetry+api@1.9.0.js	+25/-0
@prisma+instrumentation@5.22.0.js	+65/-0
@sentry+core@8.55.0.js	+1354/-0
@sentry+nextjs@8.55.0_@opentelemetry+context-async-hooks@1.30.1_@opentelemetry+core@1.30.1_@o_z3s5mh66mawwnoh5pw4lqv5chy.js	+324/-0
@sentry+node@8.55.0.js	+744/-0
@sentry+opentelemetry@8.55.0_@opentelemetry+api@1.9.0_@opentelemetry+context-async-hooks@1.30_wab2xbjb6maajk5mmcgtt34ivi.js	+24/-0
@swc+helpers@0.5.15.js	+25/-0
ansi-styles@4.3.0.js	+25/-0
chalk@3.0.0.js	+45/-0
color-convert@2.0.1.js	+44/-0
color-name@1.1.4.js	+25/-0
debug@4.4.0.js	+54/-0
forwarded-parse@2.1.2.js	+45/-0
function-bind@1.1.2.js	+35/-0
hasown@2.0.2.js	+25/-0
import-in-the-middle@1.13.0.js	+34/-0
is-core-module@2.16.1.js	+35/-0
ms@2.1.3.js	+24/-0
next@15.1.2_@babel+core@7.26.9_@opentelemetry+api@1.9.0_react-dom@18.3.1_react@18.3.1.js	+45/-0
path-parse@1.0.7.js	+25/-0
require-in-the-middle@7.5.1.js	+35/-0
resolve@1.22.8.js	+117/-0
semver@7.7.1.js	+464/-0
shimmer@1.2.1.js	+25/-0
stacktrace-parser@0.1.11.js	+25/-0
supports-color@7.2.0.js	+25/-0
webpack-runtime.js	+214/-0
polyfills.js	+1/-0
_buildManifest.js	+1/-0
_ssgManifest.js	+1/-0
trace	+4/-0
package.json	+1/-0
prism.wasm	[link]

Additional Notes

Need help?
Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
Check out the documentation for more information.

changeset-bot · 2025-03-12T20:30:10Z

⚠️ No Changeset found

Latest commit: 95f7321

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

vercel · 2025-03-12T20:30:12Z

The latest updates on your projects. Learn more about Vercel for Git ↗︎

3 Skipped Deployments

Name	Status	Updated (UTC)
test-liam-docs	⬜️ Ignored (Inspect)	Mar 13, 2025 6:22pm
test-liam-erd-sample	⬜️ Ignored (Inspect)	Mar 13, 2025 6:22pm
test-liam-erd-web	⬜️ Ignored (Inspect)	Mar 13, 2025 6:22pm

vercel · 2025-03-12T20:30:15Z

@prakha is attempting to deploy a commit to the ROUTE06 Core Team on Vercel.

A member of the Team first needs to authorize it.

qodo-merge-pro-for-open-source · 2025-03-12T20:31:15Z

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 2 🔵🔵⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review Tooltip Implementation The TableNode component now wraps its content in a tooltip, but this might cause unexpected behavior if the table is large or if users need to interact with elements inside the table. Verify that the tooltip doesn't interfere with normal table interactions. <TooltipProvider> <TooltipRoot> <TooltipTrigger asChild> <div className={clsx( styles.wrapper, data.isHighlighted && styles.wrapperHighlighted, data.isActiveHighlighted && styles.wrapperActive )} data-erd={ (data.isHighlighted \|\| data.isActiveHighlighted) && "table-node-highlighted" } > <TableHeader data={data} /> {showMode === "ALL_FIELDS" && <TableColumnList data={data} />} {showMode === "KEY_ONLY" && ( <TableColumnList data={data} filter="KEY_ONLY" /> )} </div> </TooltipTrigger> <TooltipPortal> <TooltipContent side={"top"} sideOffset={4}> {data?.table?.name} </TooltipContent> </TooltipPortal> </TooltipRoot> </TooltipProvider> Removed Tooltip The tooltip functionality was removed from TableHeader and moved to the parent TableNode component. Ensure this doesn't create duplicate tooltips when both components are used together. <span className={styles.name}>{name}</span>

qodo-merge-pro-for-open-source · 2025-03-12T20:31:35Z

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
General	Missing tooltip functionality The tooltip functionality has been removed from the TableHeader component but is now implemented at the TableNode level. This could lead to inconsistent behavior when users hover over table names in different contexts. Consider keeping the tooltip in both places or ensuring the parent component's tooltip covers all use cases. frontend/packages/erd-core/src/features/erd/components/ERDContent/components/TableNode/TableHeader/TableHeader.tsx [28-36] <div className={clsx( styles.wrapper, showMode === "TABLE_NAME" && styles.wrapperTableNameMode )} > <Table2 width={16} className={styles.tableIcon} /> - <span className={styles.name}>{name}</span> + <TooltipProvider> + <TooltipRoot> + <TooltipTrigger asChild> + <span className={styles.name}>{name}</span> + </TooltipTrigger> + <TooltipPortal> + <TooltipContent side={"top"} sideOffset={4}> + {name} + </TooltipContent> + </TooltipPortal> + </TooltipRoot> + </TooltipProvider> `[To ensure code accuracy, apply this suggestion manually]` Suggestion importance[1-10]: 8 __ Why: The suggestion correctly identifies that tooltip functionality was removed from TableHeader and moved to TableNode. This could cause inconsistent behavior in different contexts where the table name appears. Restoring the tooltip ensures consistent UX across the application.	Medium
Update

MH4GF · 2025-03-13T00:05:37Z

Hi @prakha, Can you check your changes? You seem to be committing .next directory.

junkisai · 2025-03-13T11:46:59Z

Since apps/erd-web has been renamed to apps/app, it seems necessary to incorporate the latest main branch.

…it_easier_to_display_table_names

prakha added 3 commits March 12, 2025 17:32

pull changes

cfcfaa1

removed the tooltip from the table header name

584fb96

add tooltip to TableNode

43ad3fc

prakha requested a review from a team as a code owner March 12, 2025 20:30

prakha requested review from hoshinotsuyoshi, FunamaYukina, junkisai, MH4GF and NoritakaIkeda and removed request for a team March 12, 2025 20:30

prakha had a problem deploying to preview - app March 12, 2025 20:30 — with GitHub Actions Failure

qodo-merge-pro-for-open-source bot added the Review effort 2/5 label Mar 12, 2025

removed tooltip modules from table header file

9a24d73

prakha had a problem deploying to preview - app March 12, 2025 20:35 — with GitHub Actions Failure

included .next in git ignore

047901f

prakha had a problem deploying to preview - app March 13, 2025 11:50 — with GitHub Actions Failure

Merge branch 'main' of https://github.com/prakha/liam into feat/make_…

5ca6a49

…it_easier_to_display_table_names

prakha had a problem deploying to preview - app March 13, 2025 11:56 — with GitHub Actions Failure

fix the lint issue

95f7321

prakha had a problem deploying to preview - app March 13, 2025 18:23 — with GitHub Actions Failure

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feat/make it easier to display table names #876

Feat/make it easier to display table names #876

prakha commented Mar 12, 2025 •

edited by qodo-merge-pro-for-open-source bot

Loading

changeset-bot bot commented Mar 12, 2025 •

edited

Loading

vercel bot commented Mar 12, 2025 •

edited

Loading

vercel bot commented Mar 12, 2025

qodo-merge-pro-for-open-source bot commented Mar 12, 2025

qodo-merge-pro-for-open-source bot commented Mar 12, 2025 •

edited

Loading

MH4GF commented Mar 13, 2025 •

edited

Loading

junkisai commented Mar 13, 2025 •

edited

Loading

Feat/make it easier to display table names #876

Are you sure you want to change the base?

Feat/make it easier to display table names #876

Conversation

prakha commented Mar 12, 2025 • edited by qodo-merge-pro-for-open-source bot Loading

Issue

Why is this change needed?

What would you like reviewers to focus on?

Testing Verification

What was done

🤖 Generated by PR Agent at 43ad3fc

Detailed Changes

Additional Notes

changeset-bot bot commented Mar 12, 2025 • edited Loading

⚠️ No Changeset found

vercel bot commented Mar 12, 2025 • edited Loading

vercel bot commented Mar 12, 2025

qodo-merge-pro-for-open-source bot commented Mar 12, 2025

PR Reviewer Guide 🔍

qodo-merge-pro-for-open-source bot commented Mar 12, 2025 • edited Loading

PR Code Suggestions ✨

MH4GF commented Mar 13, 2025 • edited Loading

junkisai commented Mar 13, 2025 • edited Loading

prakha commented Mar 12, 2025 •

edited by qodo-merge-pro-for-open-source bot

Loading

🤖 Generated by PR Agent at `43ad3fc`

changeset-bot bot commented Mar 12, 2025 •

edited

Loading

vercel bot commented Mar 12, 2025 •

edited

Loading

qodo-merge-pro-for-open-source bot commented Mar 12, 2025 •

edited

Loading

MH4GF commented Mar 13, 2025 •

edited

Loading

junkisai commented Mar 13, 2025 •

edited

Loading