Skip to content

refactor: simplify OptimizedImage API and enforce src/srcSet validation#7

Merged
TomChicken-on-github merged 1 commit intomainfrom
codex/refactor-optimizedimage.vue-component
Feb 24, 2026
Merged

refactor: simplify OptimizedImage API and enforce src/srcSet validation#7
TomChicken-on-github merged 1 commit intomainfrom
codex/refactor-optimizedimage.vue-component

Conversation

@TomChicken-on-github
Copy link
Copy Markdown
Collaborator

Motivation

  • 精简 OptimizedImage.vue,只保留多格式输出(<picture>)与单图回退的职责,去除与渐进加载相关的复杂状态与副作用。
  • 统一组件对外 API,移除未使用或重复的 props/emit,以降低维护成本和误用风险。
  • 增加运行时校验以确保当没有传 srcSet 时必须提供 src,从而避免在运行时出现不可预测的占位或空图像。

Description

  • 移除渐进加载实现与相关依赖:删除 useImageOptimizationgetThumbnailPath、缩略图相关 props(thumbnail)与状态、以及 load/error emit 和处理函数。
  • 保留模板中多格式支持:保留 <picture>(优先 avif/webp)和 v-else 回退 <img>,以及现有的 loading/decoding/width/height props 与样式。
  • 调整 props:将 src 设置为可选(默认 ''),保留 srcSet 可选;新增运行时校验(watchEffect)在 !srcSet && !src 时抛错以强制校验。
  • 更新注释与导入:删除未使用的导入,改为仅导入 watchEffect,并更新组件注释以反映精简后的职责。

Testing

  • 运行生产构建 npm run build 成功,构建输出与 manifest 正常生成。
  • 使用搜索验证(`rg

Codex Task

@cloudflare-workers-and-pages
Copy link
Copy Markdown

cloudflare-workers-and-pages bot commented Feb 24, 2026

Deploying wsmcs with  Cloudflare Pages  Cloudflare Pages

Latest commit: e1e29f4
Status: ✅  Deploy successful!
Preview URL: https://ffdda914.wsmcs.pages.dev
Branch Preview URL: https://codex-refactor-optimizedimag.wsmcs.pages.dev

View logs

@vercel
Copy link
Copy Markdown

vercel bot commented Feb 24, 2026

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

Project Deployment Actions Updated (UTC)
wsmcs Ready Ready Preview, Comment Feb 24, 2026 0:13am

@netlify
Copy link
Copy Markdown

netlify bot commented Feb 24, 2026

Deploy Preview for wsmcs ready!

Name Link
🔨 Latest commit e1e29f4
🔍 Latest deploy log https://app.netlify.com/projects/wsmcs/deploys/699d95d20d72500008c58b80
😎 Deploy Preview https://deploy-preview-7--wsmcs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@TomChicken-on-github TomChicken-on-github merged commit d3796e1 into main Feb 24, 2026
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant