feat(desktop): simplify employee creation configuration (#297) - #331
Merged
Merged
Conversation
Bindy-lbb
approved these changes
Sep 18, 2026
Bindy-lbb
left a comment
Collaborator
There was a problem hiding this comment.
Code Review Summary
Reviewed PR #331: feat(desktop): simplify employee creation configuration (#297)
1. Agent Icon Normalization & Optical Scaling
- Vector Asset & Quality: Replacing
workbuddy.pngwith officialworkbuddy.svgprevents raster blur on high-DPI displays. - Unified 18px Layout Box: Increasing
.owb-engine-iconfrom 14px to 18px matches the typography rhythm and makes compact brand marks legible. - Optical Scale Balance:
- Applying
--owb-engine-icon-optical-scale: 1.2for Qoder compensates for built-in transparent whitespace in its raster canvas. - Applying
--owb-engine-icon-optical-scale: 0.92for WorkBuddy prevents its full-bleed SVG from overpowering adjacent marks. - Both icons now present equivalent visual weight and area in the tree and selectors.
- Applying
- Packaging Integrity: Updating
scripts/verify-packaged-app.mjswhitelist to allow SVG assets inassets/ensures build/staging verification succeeds without false rejections.
2. Progressive Disclosure in Employee Creation Drawer
- Information Hierarchy: Moving optional Agent role drafting and advanced controls (avatar, permission policies, memory, mode, budget, capabilities) into collapsible
<details>elements substantially reduces cognitive load while retaining all defaults, validation, and payload integrity on submission. - In-Surface Accessible Choice Menus (
HireChoiceSelect): Replacing Ant Design<Select>with dedicated lightweight dropdowns resolves z-index/portal issues inside drawer contexts while maintaining ARIA attributes (aria-haspopup="listbox",aria-expanded,role="listbox",role="option",aria-selected, ESC key listener). - Localization: Both English (
en.ts) and Chinese (zh.ts) catalogs cleanly cover all new labels and hints (hire.assist*,hire.basic*,hire.advanced*,hire.choiceCurrent).
3. Verification & CI Status
npm run typecheck:uiandnpm run typecheck:rendererpassed with 0 errors.- Targeted tests in
engine-icon.test.tsxandhire-drawer.test.tsxpassed (12/12). git diff --checkpassed with no formatting anomalies.- GitHub Actions CI reports all 11 checks green.
LGTM! Approved.
This was referenced Sep 18, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Canonical requirement
This PR delivers the creation-drawer slice of the requirement and the user-requested simplification of employee setup. It does not claim completion of the unrelated recovery or model-catalog slices tracked by the same issue.
What changed
REQ / AC trace
File domains
Validation
Security / docs
Limits and rollback
Reviewer handoff
Latest icon sizing refinement