fix(skills): unbox the skills module — drop the outer card frame by jackwener · Pull Request #1185 · apache/maka

jackwener · 2026-07-18T05:55:41Z

User-reported (screenshot): the skills page wraps all its content in an outer bordered card that the MCP page doesn't have, and the page reads better without it.

The .maka-skill-library wrapper carried full card chrome (card-bg + hairline border + radius-surface + card shadow + inner padding) around the featured banner, tabs, and installed list — box-in-box, since every inner surface already has its own border, and its top edge crowded the module subtitle. Now a plain layout container (width/centering/grid/gap only), so content sits directly on the module surface flush with the header — same layout language as the MCP page.

radius-converge-contract entry for the wrapper dropped with the established unboxing comment pattern (precedent: .maka-daily-review-info). JSX untouched (skills.test.ts pins stay green as-is).

Gates: desktop 2686/2686 · ui 183/183 · typecheck · check-dead-css · knip ×2 = 0 · alignment auditor clean. CDP capture on module-skills confirms the frame is gone and content aligns with the header.

The whole module (featured banner + tabs + installed list) was wrapped in a second card (bg + hairline + radius + shadow) that the MCP page never had; inner surfaces already carry their own borders, so the wrapper read as box-in-box and its top edge crowded the module subtitle. Plain layout container now, content flush with the header like the MCP page. radius-converge entry dropped with the established unboxing note.

… language (#1187) * fix(modules): unbox 每日回顾 and 定时任务 module pages The 定时任务 (.maka-plan-shell) and 每日回顾 (.maka-module-main .maka-daily-review-panel) wrappers each carried full card chrome (card-bg + hairline + radius-surface + card-shadow + padding) around the whole module, wrapping it in a second frame the MCP page never had. Inner surfaces (system alert, task cards, run rows, archive body, stat tiles, alerts) already carry their own borders, so the wrapper read as box-in-box and crowded the module subtitle. Both are plain layout containers now: chrome stripped, padding 0, layout properties kept, daily-review width converged 980 -> 900 to track the module header measure. Content sits flush with the module header like the skills (#1185) / MCP pages. radius-converge entries for both selectors re-pinned with the established unboxing note. Follows the #1185 skills-unbox precedent. * refactor(daily-review): converge page onto the house language Presentation-only pass bringing 每日回顾 in line with the skills / MCP pages (zero behavior change — same data, actions, IPC): - Section headers: 活跃对话 / 模型使用 / 工具调用 rendered a hand-rolled uppercase eyebrow with a bespoke accent bar — the exact dialect section-header.tsx was built to retire — while 已生成报告 on the same page already used the SectionHeader primitive. All section titles now render through SectionHeader (as=h4, accent) so the page speaks one section-header language. Dropped the .maka-daily-review-section-title + ::before CSS. - Stepper: the ‹ › day-navigation controls used bare text glyphs on a ghost icon button; swapped to ChevronLeft / ChevronRight from @maka/ui/icons (icon-governance: no strokeWidth at the callsite). - Retry alert: .maka-daily-review-alert re-implemented the warning surface (flex row + hairline + tint + radius) by hand, fighting the Alert primitive's grid slot layout. Dropped the bespoke chrome; the warning variant owns border / tint / radius / padding and the description-action split. Kept .maka-daily-review-alert-retry flex:none. - Section divider token converged 0.07-alpha -> --foreground-10 to match the header / quick-runs / archives dividers on the same page.

Five debug captures from the sidebar-rail (#1180) and skills-unbox (#1185) verification runs were swept into those commits by git add -A from the worktree root. Repo root is not a screenshot store.

jackwener · 2026-07-18T05:55:41Z

User-reported (screenshot): the skills page wraps all its content in an outer bordered card that the MCP page doesn't have, and the page reads better without it.

The .maka-skill-library wrapper carried full card chrome (card-bg + hairline border + radius-surface + card shadow + inner padding) around the featured banner, tabs, and installed list — box-in-box, since every inner surface already has its own border, and its top edge crowded the module subtitle. Now a plain layout container (width/centering/grid/gap only), so content sits directly on the module surface flush with the header — same layout language as the MCP page.

radius-converge-contract entry for the wrapper dropped with the established unboxing comment pattern (precedent: .maka-daily-review-info). JSX untouched (skills.test.ts pins stay green as-is).

Gates: desktop 2686/2686 · ui 183/183 · typecheck · check-dead-css · knip ×2 = 0 · alignment auditor clean. CDP capture on module-skills confirms the frame is gone and content aligns with the header.

The whole module (featured banner + tabs + installed list) was wrapped in a second card (bg + hairline + radius + shadow) that the MCP page never had; inner surfaces already carry their own borders, so the wrapper read as box-in-box and its top edge crowded the module subtitle. Plain layout container now, content flush with the header like the MCP page. radius-converge entry dropped with the established unboxing note.

… language (#1187) * fix(modules): unbox 每日回顾 and 定时任务 module pages The 定时任务 (.maka-plan-shell) and 每日回顾 (.maka-module-main .maka-daily-review-panel) wrappers each carried full card chrome (card-bg + hairline + radius-surface + card-shadow + padding) around the whole module, wrapping it in a second frame the MCP page never had. Inner surfaces (system alert, task cards, run rows, archive body, stat tiles, alerts) already carry their own borders, so the wrapper read as box-in-box and crowded the module subtitle. Both are plain layout containers now: chrome stripped, padding 0, layout properties kept, daily-review width converged 980 -> 900 to track the module header measure. Content sits flush with the module header like the skills (#1185) / MCP pages. radius-converge entries for both selectors re-pinned with the established unboxing note. Follows the #1185 skills-unbox precedent. * refactor(daily-review): converge page onto the house language Presentation-only pass bringing 每日回顾 in line with the skills / MCP pages (zero behavior change — same data, actions, IPC): - Section headers: 活跃对话 / 模型使用 / 工具调用 rendered a hand-rolled uppercase eyebrow with a bespoke accent bar — the exact dialect section-header.tsx was built to retire — while 已生成报告 on the same page already used the SectionHeader primitive. All section titles now render through SectionHeader (as=h4, accent) so the page speaks one section-header language. Dropped the .maka-daily-review-section-title + ::before CSS. - Stepper: the ‹ › day-navigation controls used bare text glyphs on a ghost icon button; swapped to ChevronLeft / ChevronRight from @maka/ui/icons (icon-governance: no strokeWidth at the callsite). - Retry alert: .maka-daily-review-alert re-implemented the warning surface (flex row + hairline + tint + radius) by hand, fighting the Alert primitive's grid slot layout. Dropped the bespoke chrome; the warning variant owns border / tint / radius / padding and the description-action split. Kept .maka-daily-review-alert-retry flex:none. - Section divider token converged 0.07-alpha -> --foreground-10 to match the header / quick-runs / archives dividers on the same page.

Five debug captures from the sidebar-rail (#1180) and skills-unbox (#1185) verification runs were swept into those commits by git add -A from the worktree root. Repo root is not a screenshot store.

jackwener · 2026-07-18T05:55:41Z

User-reported (screenshot): the skills page wraps all its content in an outer bordered card that the MCP page doesn't have, and the page reads better without it.

The .maka-skill-library wrapper carried full card chrome (card-bg + hairline border + radius-surface + card shadow + inner padding) around the featured banner, tabs, and installed list — box-in-box, since every inner surface already has its own border, and its top edge crowded the module subtitle. Now a plain layout container (width/centering/grid/gap only), so content sits directly on the module surface flush with the header — same layout language as the MCP page.

radius-converge-contract entry for the wrapper dropped with the established unboxing comment pattern (precedent: .maka-daily-review-info). JSX untouched (skills.test.ts pins stay green as-is).

Gates: desktop 2686/2686 · ui 183/183 · typecheck · check-dead-css · knip ×2 = 0 · alignment auditor clean. CDP capture on module-skills confirms the frame is gone and content aligns with the header.

The whole module (featured banner + tabs + installed list) was wrapped in a second card (bg + hairline + radius + shadow) that the MCP page never had; inner surfaces already carry their own borders, so the wrapper read as box-in-box and its top edge crowded the module subtitle. Plain layout container now, content flush with the header like the MCP page. radius-converge entry dropped with the established unboxing note.

… language (#1187) * fix(modules): unbox 每日回顾 and 定时任务 module pages The 定时任务 (.maka-plan-shell) and 每日回顾 (.maka-module-main .maka-daily-review-panel) wrappers each carried full card chrome (card-bg + hairline + radius-surface + card-shadow + padding) around the whole module, wrapping it in a second frame the MCP page never had. Inner surfaces (system alert, task cards, run rows, archive body, stat tiles, alerts) already carry their own borders, so the wrapper read as box-in-box and crowded the module subtitle. Both are plain layout containers now: chrome stripped, padding 0, layout properties kept, daily-review width converged 980 -> 900 to track the module header measure. Content sits flush with the module header like the skills (#1185) / MCP pages. radius-converge entries for both selectors re-pinned with the established unboxing note. Follows the #1185 skills-unbox precedent. * refactor(daily-review): converge page onto the house language Presentation-only pass bringing 每日回顾 in line with the skills / MCP pages (zero behavior change — same data, actions, IPC): - Section headers: 活跃对话 / 模型使用 / 工具调用 rendered a hand-rolled uppercase eyebrow with a bespoke accent bar — the exact dialect section-header.tsx was built to retire — while 已生成报告 on the same page already used the SectionHeader primitive. All section titles now render through SectionHeader (as=h4, accent) so the page speaks one section-header language. Dropped the .maka-daily-review-section-title + ::before CSS. - Stepper: the ‹ › day-navigation controls used bare text glyphs on a ghost icon button; swapped to ChevronLeft / ChevronRight from @maka/ui/icons (icon-governance: no strokeWidth at the callsite). - Retry alert: .maka-daily-review-alert re-implemented the warning surface (flex row + hairline + tint + radius) by hand, fighting the Alert primitive's grid slot layout. Dropped the bespoke chrome; the warning variant owns border / tint / radius / padding and the description-action split. Kept .maka-daily-review-alert-retry flex:none. - Section divider token converged 0.07-alpha -> --foreground-10 to match the header / quick-runs / archives dividers on the same page.

Five debug captures from the sidebar-rail (#1180) and skills-unbox (#1185) verification runs were swept into those commits by git add -A from the worktree root. Repo root is not a screenshot store.

jackwener · 2026-07-18T05:55:41Z

User-reported (screenshot): the skills page wraps all its content in an outer bordered card that the MCP page doesn't have, and the page reads better without it.

The .maka-skill-library wrapper carried full card chrome (card-bg + hairline border + radius-surface + card shadow + inner padding) around the featured banner, tabs, and installed list — box-in-box, since every inner surface already has its own border, and its top edge crowded the module subtitle. Now a plain layout container (width/centering/grid/gap only), so content sits directly on the module surface flush with the header — same layout language as the MCP page.

radius-converge-contract entry for the wrapper dropped with the established unboxing comment pattern (precedent: .maka-daily-review-info). JSX untouched (skills.test.ts pins stay green as-is).

Gates: desktop 2686/2686 · ui 183/183 · typecheck · check-dead-css · knip ×2 = 0 · alignment auditor clean. CDP capture on module-skills confirms the frame is gone and content aligns with the header.

The whole module (featured banner + tabs + installed list) was wrapped in a second card (bg + hairline + radius + shadow) that the MCP page never had; inner surfaces already carry their own borders, so the wrapper read as box-in-box and its top edge crowded the module subtitle. Plain layout container now, content flush with the header like the MCP page. radius-converge entry dropped with the established unboxing note.

… language (#1187) * fix(modules): unbox 每日回顾 and 定时任务 module pages The 定时任务 (.maka-plan-shell) and 每日回顾 (.maka-module-main .maka-daily-review-panel) wrappers each carried full card chrome (card-bg + hairline + radius-surface + card-shadow + padding) around the whole module, wrapping it in a second frame the MCP page never had. Inner surfaces (system alert, task cards, run rows, archive body, stat tiles, alerts) already carry their own borders, so the wrapper read as box-in-box and crowded the module subtitle. Both are plain layout containers now: chrome stripped, padding 0, layout properties kept, daily-review width converged 980 -> 900 to track the module header measure. Content sits flush with the module header like the skills (#1185) / MCP pages. radius-converge entries for both selectors re-pinned with the established unboxing note. Follows the #1185 skills-unbox precedent. * refactor(daily-review): converge page onto the house language Presentation-only pass bringing 每日回顾 in line with the skills / MCP pages (zero behavior change — same data, actions, IPC): - Section headers: 活跃对话 / 模型使用 / 工具调用 rendered a hand-rolled uppercase eyebrow with a bespoke accent bar — the exact dialect section-header.tsx was built to retire — while 已生成报告 on the same page already used the SectionHeader primitive. All section titles now render through SectionHeader (as=h4, accent) so the page speaks one section-header language. Dropped the .maka-daily-review-section-title + ::before CSS. - Stepper: the ‹ › day-navigation controls used bare text glyphs on a ghost icon button; swapped to ChevronLeft / ChevronRight from @maka/ui/icons (icon-governance: no strokeWidth at the callsite). - Retry alert: .maka-daily-review-alert re-implemented the warning surface (flex row + hairline + tint + radius) by hand, fighting the Alert primitive's grid slot layout. Dropped the bespoke chrome; the warning variant owns border / tint / radius / padding and the description-action split. Kept .maka-daily-review-alert-retry flex:none. - Section divider token converged 0.07-alpha -> --foreground-10 to match the header / quick-runs / archives dividers on the same page.

Five debug captures from the sidebar-rail (#1180) and skills-unbox (#1185) verification runs were swept into those commits by git add -A from the worktree root. Repo root is not a screenshot store.

jackwener · 2026-07-18T05:55:41Z

User-reported (screenshot): the skills page wraps all its content in an outer bordered card that the MCP page doesn't have, and the page reads better without it.

The .maka-skill-library wrapper carried full card chrome (card-bg + hairline border + radius-surface + card shadow + inner padding) around the featured banner, tabs, and installed list — box-in-box, since every inner surface already has its own border, and its top edge crowded the module subtitle. Now a plain layout container (width/centering/grid/gap only), so content sits directly on the module surface flush with the header — same layout language as the MCP page.

radius-converge-contract entry for the wrapper dropped with the established unboxing comment pattern (precedent: .maka-daily-review-info). JSX untouched (skills.test.ts pins stay green as-is).

Gates: desktop 2686/2686 · ui 183/183 · typecheck · check-dead-css · knip ×2 = 0 · alignment auditor clean. CDP capture on module-skills confirms the frame is gone and content aligns with the header.

The whole module (featured banner + tabs + installed list) was wrapped in a second card (bg + hairline + radius + shadow) that the MCP page never had; inner surfaces already carry their own borders, so the wrapper read as box-in-box and its top edge crowded the module subtitle. Plain layout container now, content flush with the header like the MCP page. radius-converge entry dropped with the established unboxing note.

… language (#1187) * fix(modules): unbox 每日回顾 and 定时任务 module pages The 定时任务 (.maka-plan-shell) and 每日回顾 (.maka-module-main .maka-daily-review-panel) wrappers each carried full card chrome (card-bg + hairline + radius-surface + card-shadow + padding) around the whole module, wrapping it in a second frame the MCP page never had. Inner surfaces (system alert, task cards, run rows, archive body, stat tiles, alerts) already carry their own borders, so the wrapper read as box-in-box and crowded the module subtitle. Both are plain layout containers now: chrome stripped, padding 0, layout properties kept, daily-review width converged 980 -> 900 to track the module header measure. Content sits flush with the module header like the skills (#1185) / MCP pages. radius-converge entries for both selectors re-pinned with the established unboxing note. Follows the #1185 skills-unbox precedent. * refactor(daily-review): converge page onto the house language Presentation-only pass bringing 每日回顾 in line with the skills / MCP pages (zero behavior change — same data, actions, IPC): - Section headers: 活跃对话 / 模型使用 / 工具调用 rendered a hand-rolled uppercase eyebrow with a bespoke accent bar — the exact dialect section-header.tsx was built to retire — while 已生成报告 on the same page already used the SectionHeader primitive. All section titles now render through SectionHeader (as=h4, accent) so the page speaks one section-header language. Dropped the .maka-daily-review-section-title + ::before CSS. - Stepper: the ‹ › day-navigation controls used bare text glyphs on a ghost icon button; swapped to ChevronLeft / ChevronRight from @maka/ui/icons (icon-governance: no strokeWidth at the callsite). - Retry alert: .maka-daily-review-alert re-implemented the warning surface (flex row + hairline + tint + radius) by hand, fighting the Alert primitive's grid slot layout. Dropped the bespoke chrome; the warning variant owns border / tint / radius / padding and the description-action split. Kept .maka-daily-review-alert-retry flex:none. - Section divider token converged 0.07-alpha -> --foreground-10 to match the header / quick-runs / archives dividers on the same page.

Five debug captures from the sidebar-rail (#1180) and skills-unbox (#1185) verification runs were swept into those commits by git add -A from the worktree root. Repo root is not a screenshot store.

jackwener · 2026-07-18T05:55:41Z

User-reported (screenshot): the skills page wraps all its content in an outer bordered card that the MCP page doesn't have, and the page reads better without it.

The .maka-skill-library wrapper carried full card chrome (card-bg + hairline border + radius-surface + card shadow + inner padding) around the featured banner, tabs, and installed list — box-in-box, since every inner surface already has its own border, and its top edge crowded the module subtitle. Now a plain layout container (width/centering/grid/gap only), so content sits directly on the module surface flush with the header — same layout language as the MCP page.

radius-converge-contract entry for the wrapper dropped with the established unboxing comment pattern (precedent: .maka-daily-review-info). JSX untouched (skills.test.ts pins stay green as-is).

Gates: desktop 2686/2686 · ui 183/183 · typecheck · check-dead-css · knip ×2 = 0 · alignment auditor clean. CDP capture on module-skills confirms the frame is gone and content aligns with the header.

The whole module (featured banner + tabs + installed list) was wrapped in a second card (bg + hairline + radius + shadow) that the MCP page never had; inner surfaces already carry their own borders, so the wrapper read as box-in-box and its top edge crowded the module subtitle. Plain layout container now, content flush with the header like the MCP page. radius-converge entry dropped with the established unboxing note.

… language (#1187) * fix(modules): unbox 每日回顾 and 定时任务 module pages The 定时任务 (.maka-plan-shell) and 每日回顾 (.maka-module-main .maka-daily-review-panel) wrappers each carried full card chrome (card-bg + hairline + radius-surface + card-shadow + padding) around the whole module, wrapping it in a second frame the MCP page never had. Inner surfaces (system alert, task cards, run rows, archive body, stat tiles, alerts) already carry their own borders, so the wrapper read as box-in-box and crowded the module subtitle. Both are plain layout containers now: chrome stripped, padding 0, layout properties kept, daily-review width converged 980 -> 900 to track the module header measure. Content sits flush with the module header like the skills (#1185) / MCP pages. radius-converge entries for both selectors re-pinned with the established unboxing note. Follows the #1185 skills-unbox precedent. * refactor(daily-review): converge page onto the house language Presentation-only pass bringing 每日回顾 in line with the skills / MCP pages (zero behavior change — same data, actions, IPC): - Section headers: 活跃对话 / 模型使用 / 工具调用 rendered a hand-rolled uppercase eyebrow with a bespoke accent bar — the exact dialect section-header.tsx was built to retire — while 已生成报告 on the same page already used the SectionHeader primitive. All section titles now render through SectionHeader (as=h4, accent) so the page speaks one section-header language. Dropped the .maka-daily-review-section-title + ::before CSS. - Stepper: the ‹ › day-navigation controls used bare text glyphs on a ghost icon button; swapped to ChevronLeft / ChevronRight from @maka/ui/icons (icon-governance: no strokeWidth at the callsite). - Retry alert: .maka-daily-review-alert re-implemented the warning surface (flex row + hairline + tint + radius) by hand, fighting the Alert primitive's grid slot layout. Dropped the bespoke chrome; the warning variant owns border / tint / radius / padding and the description-action split. Kept .maka-daily-review-alert-retry flex:none. - Section divider token converged 0.07-alpha -> --foreground-10 to match the header / quick-runs / archives dividers on the same page.

Five debug captures from the sidebar-rail (#1180) and skills-unbox (#1185) verification runs were swept into those commits by git add -A from the worktree root. Repo root is not a screenshot store.

jackwener · 2026-07-18T05:55:41Z

User-reported (screenshot): the skills page wraps all its content in an outer bordered card that the MCP page doesn't have, and the page reads better without it.

The .maka-skill-library wrapper carried full card chrome (card-bg + hairline border + radius-surface + card shadow + inner padding) around the featured banner, tabs, and installed list — box-in-box, since every inner surface already has its own border, and its top edge crowded the module subtitle. Now a plain layout container (width/centering/grid/gap only), so content sits directly on the module surface flush with the header — same layout language as the MCP page.

radius-converge-contract entry for the wrapper dropped with the established unboxing comment pattern (precedent: .maka-daily-review-info). JSX untouched (skills.test.ts pins stay green as-is).

Gates: desktop 2686/2686 · ui 183/183 · typecheck · check-dead-css · knip ×2 = 0 · alignment auditor clean. CDP capture on module-skills confirms the frame is gone and content aligns with the header.

The whole module (featured banner + tabs + installed list) was wrapped in a second card (bg + hairline + radius + shadow) that the MCP page never had; inner surfaces already carry their own borders, so the wrapper read as box-in-box and its top edge crowded the module subtitle. Plain layout container now, content flush with the header like the MCP page. radius-converge entry dropped with the established unboxing note.

… language (#1187) * fix(modules): unbox 每日回顾 and 定时任务 module pages The 定时任务 (.maka-plan-shell) and 每日回顾 (.maka-module-main .maka-daily-review-panel) wrappers each carried full card chrome (card-bg + hairline + radius-surface + card-shadow + padding) around the whole module, wrapping it in a second frame the MCP page never had. Inner surfaces (system alert, task cards, run rows, archive body, stat tiles, alerts) already carry their own borders, so the wrapper read as box-in-box and crowded the module subtitle. Both are plain layout containers now: chrome stripped, padding 0, layout properties kept, daily-review width converged 980 -> 900 to track the module header measure. Content sits flush with the module header like the skills (#1185) / MCP pages. radius-converge entries for both selectors re-pinned with the established unboxing note. Follows the #1185 skills-unbox precedent. * refactor(daily-review): converge page onto the house language Presentation-only pass bringing 每日回顾 in line with the skills / MCP pages (zero behavior change — same data, actions, IPC): - Section headers: 活跃对话 / 模型使用 / 工具调用 rendered a hand-rolled uppercase eyebrow with a bespoke accent bar — the exact dialect section-header.tsx was built to retire — while 已生成报告 on the same page already used the SectionHeader primitive. All section titles now render through SectionHeader (as=h4, accent) so the page speaks one section-header language. Dropped the .maka-daily-review-section-title + ::before CSS. - Stepper: the ‹ › day-navigation controls used bare text glyphs on a ghost icon button; swapped to ChevronLeft / ChevronRight from @maka/ui/icons (icon-governance: no strokeWidth at the callsite). - Retry alert: .maka-daily-review-alert re-implemented the warning surface (flex row + hairline + tint + radius) by hand, fighting the Alert primitive's grid slot layout. Dropped the bespoke chrome; the warning variant owns border / tint / radius / padding and the description-action split. Kept .maka-daily-review-alert-retry flex:none. - Section divider token converged 0.07-alpha -> --foreground-10 to match the header / quick-runs / archives dividers on the same page.

Five debug captures from the sidebar-rail (#1180) and skills-unbox (#1185) verification runs were swept into those commits by git add -A from the worktree root. Repo root is not a screenshot store.

jackwener · 2026-07-18T05:55:41Z

User-reported (screenshot): the skills page wraps all its content in an outer bordered card that the MCP page doesn't have, and the page reads better without it.

The .maka-skill-library wrapper carried full card chrome (card-bg + hairline border + radius-surface + card shadow + inner padding) around the featured banner, tabs, and installed list — box-in-box, since every inner surface already has its own border, and its top edge crowded the module subtitle. Now a plain layout container (width/centering/grid/gap only), so content sits directly on the module surface flush with the header — same layout language as the MCP page.

radius-converge-contract entry for the wrapper dropped with the established unboxing comment pattern (precedent: .maka-daily-review-info). JSX untouched (skills.test.ts pins stay green as-is).

Gates: desktop 2686/2686 · ui 183/183 · typecheck · check-dead-css · knip ×2 = 0 · alignment auditor clean. CDP capture on module-skills confirms the frame is gone and content aligns with the header.

The whole module (featured banner + tabs + installed list) was wrapped in a second card (bg + hairline + radius + shadow) that the MCP page never had; inner surfaces already carry their own borders, so the wrapper read as box-in-box and its top edge crowded the module subtitle. Plain layout container now, content flush with the header like the MCP page. radius-converge entry dropped with the established unboxing note.

… language (#1187) * fix(modules): unbox 每日回顾 and 定时任务 module pages The 定时任务 (.maka-plan-shell) and 每日回顾 (.maka-module-main .maka-daily-review-panel) wrappers each carried full card chrome (card-bg + hairline + radius-surface + card-shadow + padding) around the whole module, wrapping it in a second frame the MCP page never had. Inner surfaces (system alert, task cards, run rows, archive body, stat tiles, alerts) already carry their own borders, so the wrapper read as box-in-box and crowded the module subtitle. Both are plain layout containers now: chrome stripped, padding 0, layout properties kept, daily-review width converged 980 -> 900 to track the module header measure. Content sits flush with the module header like the skills (#1185) / MCP pages. radius-converge entries for both selectors re-pinned with the established unboxing note. Follows the #1185 skills-unbox precedent. * refactor(daily-review): converge page onto the house language Presentation-only pass bringing 每日回顾 in line with the skills / MCP pages (zero behavior change — same data, actions, IPC): - Section headers: 活跃对话 / 模型使用 / 工具调用 rendered a hand-rolled uppercase eyebrow with a bespoke accent bar — the exact dialect section-header.tsx was built to retire — while 已生成报告 on the same page already used the SectionHeader primitive. All section titles now render through SectionHeader (as=h4, accent) so the page speaks one section-header language. Dropped the .maka-daily-review-section-title + ::before CSS. - Stepper: the ‹ › day-navigation controls used bare text glyphs on a ghost icon button; swapped to ChevronLeft / ChevronRight from @maka/ui/icons (icon-governance: no strokeWidth at the callsite). - Retry alert: .maka-daily-review-alert re-implemented the warning surface (flex row + hairline + tint + radius) by hand, fighting the Alert primitive's grid slot layout. Dropped the bespoke chrome; the warning variant owns border / tint / radius / padding and the description-action split. Kept .maka-daily-review-alert-retry flex:none. - Section divider token converged 0.07-alpha -> --foreground-10 to match the header / quick-runs / archives dividers on the same page.

Five debug captures from the sidebar-rail (#1180) and skills-unbox (#1185) verification runs were swept into those commits by git add -A from the worktree root. Repo root is not a screenshot store.

jackwener merged commit fcdf7a9 into mainJul 18, 2026
3 checks passed

jackwener deleted the fix/skills-page-unbox branch July 18, 2026 05:55

jackwener mentioned this pull request Jul 18, 2026
fix(modules): unbox 每日回顾 + 定时任务, converge daily-review onto the house language #1187
Merged

jackwener mentioned this pull request Jul 18, 2026
chore: remove stray CDP debug screenshots #1188
Merged