diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS new file mode 100644 index 0000000..2332a7c --- /dev/null +++ b/.github/CODEOWNERS @@ -0,0 +1,37 @@ +# CODEOWNERS — 善缘丹阳公益婚恋平台 审查角色分配(草案) +# +# 作用:开 PR 时 GitHub 自动向对应路径的 owner 请求审查;配合分支保护与 +# docs/governance/CODE_REVIEW_STANDARDS.md 的 T0/T1/T2 分级与双审规则使用。 +# 语法:https://docs.github.com/en/repositories/managing-your-repositories-settings-and-features/customizing-your-repository/about-code-owners +# +# 当前为一人团队,默认全部归技术负责人 @g1307092382。 +# 待扩员时,把各端拆给对应熟悉者即可;T2 高危改动按审查标准须 2 人复核(其中 1 人为该端 owner)。 +# 说明:CODEOWNERS 只负责「请求谁审」,不强制「必须不同人」;双审由审查流程纪律保证。 +# 排序:子路径必须放在父路径之后(GitHub 取最后一个匹配的规则)。 + +# —— 默认兜底 —— +* @g1307092382 + +# —— 后端(Spring Boot / Java)—— +/backend/ @g1307092382 + +# 数据库迁移(Flyway):schema 变更属 T2,须重点复核(放在 /backend/ 之后) +/backend/src/main/resources/db/migration/ @g1307092382 + +# —— 微信小程序(原生 WXML/WXSS/TS)—— +/miniapp/ @g1307092382 + +# —— 管理端(Vue3 / TS / pnpm)—— +/admin-web/ @g1307092382 + +# —— 测试与契约守卫(跨端契约,需熟悉各端)—— +/tests/ @g1307092382 +/ui-tests/ @g1307092382 +/scripts/run-ui-contracts.mjs @g1307092382 + +# —— 基础设施 / 部署(Compose / 镜像 / 密钥)—— +/infrastructure/ @g1307092382 + +# —— 治理与审查机制文档(变更须经技术负责人确认)—— +/docs/governance/ @g1307092382 +/.github/ @g1307092382 diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md new file mode 100644 index 0000000..dcb486b --- /dev/null +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -0,0 +1,69 @@ + + +## 变更摘要 + + +## 风险分级(见审查标准 §2) +- [ ] T0 琐碎(纯文案/文档/格式化/资源重命名) +- [ ] T1 常规(单端功能/Bug/UI/单 API) +- [ ] T2 高危(鉴权/支付/隐私/跨端契约/DB schema/并发幂等/已知阻塞项)— 需 2 人复核 + +## 验证方式(必填,证据优先) + +- 本地测试/契约守卫命令与结果: +- 运行时实测(如涉及渲染/接口,贴 mock/路由/关键响应): +- 负向注入(新增或修改测试时必填:故意改坏后测试是否转红): + +## 技术栈审查清单(按本 PR 涉及项打勾) +### 后端 Java / Spring Boot +- [ ] 业务拒绝用 `BusinessException`+`ApiResponseCode`(HTTP 400≤code<500),无裸 `RuntimeException` +- [ ] 测试对业务拒绝断言 `400 ≤ status < 500`(非仅 `!= 200`) +- [ ] 核心不变量(INV-014/015、Required/Mutual/Block 等)未被绕过 +- [ ] 主键/唯一键/CHECK 约束与 Java 枚举一致,无静默 no-op 写 +- [ ] 无新增死表/死配置(`system_configs`/`match_rule_configs` 类) + +### 小程序 miniapp +- [ ] `app.json`/`pages.json` 路径与磁盘文件一致、无前导 `/` +- [ ] `ignoreDevUnusedFiles`/`ignoreUploadUnusedFiles` = false(资源不被吞) +- [ ] 跨端字段名与后端规范名一致(tsc 不保护 JS 层字段名) +- [ ] WXSS 无通配符/属性选择器/嵌套注释;`` 显式高度 +- [ ] 胶囊安全区让出 `--nav-right` + +### 管理端 admin-web +- [ ] 渲染相关状态走 `ref`/`reactive`,已补渲染态实测 +- [ ] 颜色/字号收敛到 `src/theme/` token,EP 主色阶补全 `-light-N`/`-dark-2`/`-rgb` +- [ ] 表单校验类型安全、出错保留输入 +- [ ] 触摸目标 ≥44px,尊重 `prefers-reduced-motion` + +### 数据库迁移 SQL +- [ ] `V<序号>__<描述>.sql` 命名,版本连续唯一 +- [ ] 向后兼容,NOT NULL 新列有默认值/迁移 +- [ ] CHECK 约束与枚举一致,破坏性变更有回滚方案 + +### 测试 / 契约守卫 +- [ ] 无空壳 `skip`/`pass` 测试 +- [ ] 已做负向注入证明判别力 +- [ ] `scripts/run-ui-contracts.mjs` 的 `MIN_GUARDS`(当前 23) 同步(增删守卫必改) + +### 脚本 / 基础设施 +- [ ] 脚本真实依赖已在 CI 对应 job 安装(非仅本地能跑) +- [ ] 密钥仅运行期、不落盘;本地栈固定 `-p danyang` + +## 判据纪律自查(见审查标准 §5,对应改动必查) +- [ ] 非空数据验证(无空数据假绿,如推荐/列表返回空也能过) +- [ ] 无直连绕过(关键路径至少一个经 HTTP 层断言) +- [ ] 约束相关改动在带约束环境验证(无测试库缺约束假绿) +- [ ] 小程序改动不走 `Mini Program Compile Check` 的"仅 tsc"假绿,已真编译/查资源 +- [ ] 合并后回读 `git rev-parse HEAD` + 轮询 CI 确认本次 head 真 success + +## 已知阻塞 / 决策关联 +- [ ] 本 PR 是否触碰上线阻塞项(微信真实登录 / 照片直传落库)?若是:✅本次解决 / ⚠️未解决已登记 known-risks +- [ ] 是否与 `decision-register` 的 LOCKED 决策冲突?若是:已回决策层确认(附结论) +- [ ] 是否产生新债务?若是:已登记 `known-risks.md`(编号 KR-xx) + +## 关联 + diff --git a/VERSION b/VERSION index 6e28fd6..9084fa2 100644 --- a/VERSION +++ b/VERSION @@ -1 +1 @@ -1.1.0-RC.1 +1.1.0 diff --git a/admin-web/package.json b/admin-web/package.json index d0120e8..a93468e 100644 --- a/admin-web/package.json +++ b/admin-web/package.json @@ -1,6 +1,6 @@ { "name": "danyang-match-admin", - "version": "1.0.0-rc.2", + "version": "1.1.0", "private": true, "type": "module", "scripts": { diff --git a/admin-web/pnpm-workspace.yaml b/admin-web/pnpm-workspace.yaml index 2a7b9ba..4cc8c79 100644 --- a/admin-web/pnpm-workspace.yaml +++ b/admin-web/pnpm-workspace.yaml @@ -19,6 +19,9 @@ # 后者生成 Vue 版本桥接),允许它们执行。 packages: - '.' +allowBuilds: + esbuild: set this to true or false + vue-demi: set this to true or false onlyBuiltDependencies: - esbuild - vue-demi diff --git a/admin-web/src/api/client.ts b/admin-web/src/api/client.ts index a8893bd..1eb6889 100644 --- a/admin-web/src/api/client.ts +++ b/admin-web/src/api/client.ts @@ -1,14 +1,18 @@ import axios from 'axios' import { ElMessage } from 'element-plus' import router from '@/router' +import { getAccessToken, setAccessToken, clearAccessToken } from '@/utils/token' const client = axios.create({ baseURL: '/api/v1', timeout: 10000, }) +// 防止 401 时并发触发多次刷新 +let refreshing = false + client.interceptors.request.use((config) => { - const token = localStorage.getItem('accessToken') + const token = getAccessToken() if (token) { config.headers.Authorization = `Bearer ${token}` } @@ -17,11 +21,45 @@ client.interceptors.request.use((config) => { client.interceptors.response.use( (res) => res.data, - (err) => { - if (err.response?.status === 401) { - localStorage.removeItem('accessToken') - router.push('/login') + async (err) => { + const status = err.response?.status + const original = err.config + const isAuthEndpoint = !!original?.url?.includes('/auth/operator/') + + // access token 过期:尝试用 HttpOnly refresh Cookie 静默刷新一次后重试原请求 + // __retried 防止刷新后重试仍 401 时再次触发刷新,形成循环 + if (status === 401 && !refreshing && !isAuthEndpoint && !(original as any)?.__retried) { + refreshing = true + try { + const resp: any = await client.post('/auth/operator/refresh', {}) + if (resp?.success) { + const newToken = resp.data.accessToken + setAccessToken(newToken) + refreshing = false + ;(original as any).__retried = true + const headers = (original.headers || {}) as Record + headers.Authorization = `Bearer ${newToken}` + original.headers = headers as any + return client(original) + } + } catch { + // 刷新失败:清除并跳登录 + } finally { + refreshing = false + } + clearAccessToken() ElMessage.error('登录已过期,请重新登录') + router.push('/login') + return Promise.reject(err) + } + + // 刷新/登录等鉴权端点自身 401:交由调用方处理文案,不全局弹窗跳转 + if (status === 401) { + if (!isAuthEndpoint) { + clearAccessToken() + ElMessage.error('登录已过期,请重新登录') + router.push('/login') + } } else if (err.response?.data?.message) { ElMessage.error(err.response.data.message) } else { diff --git a/admin-web/src/modules/activity/ActivityCheckinView.vue b/admin-web/src/modules/activity/ActivityCheckinView.vue index defcf01..873cd95 100644 --- a/admin-web/src/modules/activity/ActivityCheckinView.vue +++ b/admin-web/src/modules/activity/ActivityCheckinView.vue @@ -80,6 +80,7 @@