Skip to content

chore: updated the playground import map to include the idb library via CDN#345

Merged
gene9831 merged 2 commits into
opentiny:developfrom
SonyLeo:feat/add-idb-import-map
May 22, 2026
Merged

chore: updated the playground import map to include the idb library via CDN#345
gene9831 merged 2 commits into
opentiny:developfrom
SonyLeo:feat/add-idb-import-map

Conversation

@SonyLeo
Copy link
Copy Markdown
Collaborator

@SonyLeo SonyLeo commented May 19, 2026

修复缺少依赖导致的 playground 案例无法正常显示

修改前:

image

修改后:

image

Summary by CodeRabbit

  • Chores
    • Updated the playground import map to include the idb library (version 8.0.3) via CDN.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 19, 2026

Walkthrough

The pull request adds a single module import mapping to the playground's import map utility. The idb package at version 8.0.3 is now configured as an available ESM module via CDN, making it accessible within the playground environment.

Changes

IDB Module Import Mapping

Layer / File(s) Summary
Add idb module to import map
packages/playground/src/utils/import-map.ts
The generateImportMap function now includes an idb@8.0.3 entry in the imports map, pointing to an ESM CDN URL.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~1 minute

Poem

🐰 A tiny hop to libraries new,
where idb awaits the playground's crew,
version eight-point-oh now in sight,
ESM imports burning bright! ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (4 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Title check ✅ Passed The title accurately describes the main change: adding the idb library to the import map in the playground, which aligns with the file-level change and PR objective to resolve missing dependency.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Comment thread packages/playground/src/utils/import-map.ts Outdated
@gene9831
Copy link
Copy Markdown
Collaborator

标题改一下

@SonyLeo SonyLeo changed the title feat: add IDB package to import map chore: updated the playground import map to include the idb library via CDN May 22, 2026
@SonyLeo
Copy link
Copy Markdown
Collaborator Author

SonyLeo commented May 22, 2026

标题改一下

好的,已修改为:

chore: updated the playground import map to include the idb library via CDN

@gene9831 gene9831 merged commit b555a43 into opentiny:develop May 22, 2026
4 checks passed
@github-actions
Copy link
Copy Markdown
Contributor

🧹 Preview Cleaned Up

The preview deployment has been removed.

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.

2 participants