Nhánh feature/perf-ui-logic: tối ưu hiệu năng/UI, sửa lỗi workspace và điều hướng, và làm cho công tắc "Block network for agent-run commands" chặn thật mọi đường ra mạng của app, trừ nhà cung cấp AI.
Bộ kiểm tra chung application/network/network_guard.py, nối vào cấu hình đang chạy ở Composition Root: đổi công tắc trong Settings là có hiệu lực ngay.
Lệnh shell của agent và task script chạy trong Windows AppContainer không có quyền mạng: kernel chặn socket, ping, DNS, Invoke-WebRequest… Không cần quyền admin. Không cô lập được thì lệnh bị từ chối, không chạy khi mạng còn mở. macOS dùng sandbox-exec, Linux dùng unshare --net.
Bật chặn thì: dừng MCP đang chạy, không khởi động server mới, từ chối lời gọi connector; Microsoft 365 (đăng nhập, Graph, đồng bộ cloud, rules, mail), Teams, nút Test REST/Jira/MCP, link đính kèm task, pip tự cài và tài nguyên web trong xem trước HTML đều bị từ chối.
Vẫn dùng được: chat, tải danh sách model, thử model; tool OneDrive đã đồng bộ trên máy.
Công tắc mặc định tắt khi mở app lần đầu; nhãn giữ nguyên như cũ.
Xem trước HTML trong tab Folder giờ hiện được ảnh/CSS/JS từ web khi mạng mở (trước đây trang file:// không tải được).
Sửa lỗi app văng khi chuyển tab Graph → Folder: profile WebEngine của trang xem trước bị huỷ trước trang (0xc0000409 trong Qt6Core.dll); giờ dùng một profile chung thuộc QApplication.
Không cấp quyền AppContainer kế thừa lên thư mục chứa PySide6 (nếu có, Chromium không nạp được Qt6WebEngineCore.dll và tab Graph trắng).
Cột mục lục trong Settings tính độ rộng theo kiểu chữ của mục đang chọn, "Sandbox Security Layer" không còn bị cắt.
Các commit khác trong nhánh
b7a41b3 mỗi thư mục làm việc chỉ thuộc về một project · bbdf146 bật nút Sửa project khi đã có project đang mở
35f24e0, cc8d5c8, 2e3e719, c699beb canh hàng / khoảng cách thanh điều hướng
2759ed9 không refresh workspace khi chuyển tab Cowork · 7607f44 checkpoint hiệu năng và UI
8548c1e chặn tool mạng của agent · caf3b74 renderer GraphRAG native trên macOS · c00b83c khoảng cách metadata hàng project · a04f8a9 ẩn picker workspace cloud
Mọi đường ra mạng do app tự mở, trừ nhà cung cấp AI (xem Summary).
Test: tests/test_network_guard_lanes.py (có bài chạy AppContainer thật trên Windows), tests/ui/test_html_preview_remote_images.py.
What is intentionally NOT included?
Chặn cả nhà cung cấp AI / chạy model trên máy (Phương án 2).
Terminal người dùng tự gõ trong tab Folder, sinh ảnh, cơ chế tự tin chứng chỉ lạ (tls_trust).
Huy hiệu trạng thái "đang chặn" trên thanh trên cùng.
Validation
Unit tests
Integration tests
Manual verification
Regression check
Commands / evidence:
python -m pytest tests/test_network_guard_lanes.py tests/test_sandbox_block_network.py tests/ui -q → chỉ còn 1 bài fail, fail cả trên b7a41b3 (nhãn ProjectRow 'Project' chưa dịch, tests/ui/test_i18n_khong_con_chu_cu.py).
python -m pytest tests -q --ignore=tests/ui → 4 bài fail, cả 4 cũng fail trên b7a41b3 (test_canonical_audit_logger, 2 bài test_mcp_audit_security, test_monitoring_tab_container).
Chạy cả tests trong một lượt thì treo ở các test dựng MainWindow trong tests/ui; b7a41b3 cũng treo đúng chỗ đó.
check_imports.py và check_orphan_modules.py PASS. check_loc.py báo 9 file quá dài, giống hệt trước khi sửa (không file nào do nhánh này làm dài thêm).
Kiểm tra tay trên Windows 11: trong AppContainer, Python báo WinError 10013, ping/nslookup/PowerShell/curl đều không ra được mạng; cmd, git, python chạy bình thường.
Kiểm tra tay trên Windows 11: xem trước HTML tải được 4/4 tài nguyên web khi mạng mở, 0/4 khi bật chặn; tab Graph hoạt động; tạo/huỷ trang xem trước nhiều lần không còn cảnh báo profile của Qt.
Security Impact
Permission / credential / network / customer data impact:
Network: khi bật công tắc, chỉ nhà cung cấp AI còn ra mạng; nội dung chat vẫn gửi tới nhà cung cấp AI.
Permission: lần đầu chạy lệnh trong sandbox, app thêm quyền (ACE) cho SID AppContainer trên thư mục làm việc (ghi), thư mục cài Python gốc (đọc), gốc venv và Scripts (đọc). Không xoá quyền nào. Thư mục chứa PySide6 không bao giờ nhận quyền kế thừa; một quyền kế thừa sai trên venv (từ bản dev trước) được tự gỡ.
Credential: không đổi. Khi chặn, trạng thái đăng nhập M365 được đọc thẳng từ kho token trên máy, không dựng MSAL.
Compatibility
No breaking change
Breaking change documented
Ghi chú: block_network mặc định đổi từ bật sang tắt cho cấu hình mới; máy đã lưu true thì giữ nguyên. Khi đang chặn, lệnh dùng công cụ cài trong thư mục người dùng (ngoài Program Files) có thể báo Access denied; thư viện trong venv của app không dùng được trong sandbox.
Reviewer Notes
infrastructure/sandbox/appcontainer_process.py gọi Win32 bằng ctypes (CreateAppContainerProfile, CreateProcessW với SECURITY_CAPABILITIES) và dùng icacls để cấp quyền: nên xem kỹ phần cấp quyền.
tests/conftest.py thêm fixture autouse gỡ network_guard sau mỗi test, vì build_context() gắn cổng này ở mức process.
core/task_executors.py đang đúng bằng trần LOC nên _run_script được tách sang core/task_script.py.
## Summary
Nhánh `feature/perf-ui-logic`: tối ưu hiệu năng/UI, sửa lỗi workspace và điều hướng, và làm cho công tắc **"Block network for agent-run commands"** chặn thật mọi đường ra mạng của app, **trừ nhà cung cấp AI**.
**Chặn mạng (b78d483, 8c497cf, 10b8379)**
- Bộ kiểm tra chung `application/network/network_guard.py`, nối vào cấu hình đang chạy ở Composition Root: đổi công tắc trong Settings là có hiệu lực ngay.
- Lệnh shell của agent và task script chạy trong **Windows AppContainer không có quyền mạng**: kernel chặn socket, ping, DNS, Invoke-WebRequest… Không cần quyền admin. Không cô lập được thì lệnh bị từ chối, không chạy khi mạng còn mở. macOS dùng `sandbox-exec`, Linux dùng `unshare --net`.
- Bật chặn thì: dừng MCP đang chạy, không khởi động server mới, từ chối lời gọi connector; Microsoft 365 (đăng nhập, Graph, đồng bộ cloud, rules, mail), Teams, nút Test REST/Jira/MCP, link đính kèm task, pip tự cài và tài nguyên web trong xem trước HTML đều bị từ chối.
- Vẫn dùng được: chat, tải danh sách model, thử model; tool OneDrive đã đồng bộ trên máy.
- Công tắc **mặc định tắt** khi mở app lần đầu; nhãn giữ nguyên như cũ.
- Xem trước HTML trong tab Folder giờ hiện được ảnh/CSS/JS từ web khi mạng mở (trước đây trang `file://` không tải được).
- Sửa lỗi app văng khi chuyển tab Graph → Folder: profile WebEngine của trang xem trước bị huỷ trước trang (`0xc0000409` trong Qt6Core.dll); giờ dùng một profile chung thuộc QApplication.
- Không cấp quyền AppContainer kế thừa lên thư mục chứa PySide6 (nếu có, Chromium không nạp được `Qt6WebEngineCore.dll` và tab Graph trắng).
- Cột mục lục trong Settings tính độ rộng theo kiểu chữ của mục đang chọn, "Sandbox Security Layer" không còn bị cắt.
**Các commit khác trong nhánh**
- `b7a41b3` mỗi thư mục làm việc chỉ thuộc về một project · `bbdf146` bật nút Sửa project khi đã có project đang mở
- `35f24e0`, `cc8d5c8`, `2e3e719`, `c699beb` canh hàng / khoảng cách thanh điều hướng
- `2759ed9` không refresh workspace khi chuyển tab Cowork · `7607f44` checkpoint hiệu năng và UI
- `8548c1e` chặn tool mạng của agent · `caf3b74` renderer GraphRAG native trên macOS · `c00b83c` khoảng cách metadata hàng project · `a04f8a9` ẩn picker workspace cloud
## Change Type
- [x] Cowork feature
- [x] Bug fix
- [ ] Core AI contribution
- [x] Test / hardening
- [x] Performance
- [ ] Documentation
## Related Work
Cowork Task:
Core Repo: http://34.143.229.138/gitea-admin/fsg-ai-core-assets
Core AI Issue:
Core Task:
Related PR:
## Scope
What is intentionally included?
- Mọi đường ra mạng do app tự mở, trừ nhà cung cấp AI (xem Summary).
- Test: `tests/test_network_guard_lanes.py` (có bài chạy AppContainer thật trên Windows), `tests/ui/test_html_preview_remote_images.py`.
What is intentionally NOT included?
- Chặn cả nhà cung cấp AI / chạy model trên máy (Phương án 2).
- Terminal người dùng tự gõ trong tab Folder, sinh ảnh, cơ chế tự tin chứng chỉ lạ (`tls_trust`).
- Huy hiệu trạng thái "đang chặn" trên thanh trên cùng.
## Validation
- [x] Unit tests
- [x] Integration tests
- [x] Manual verification
- [x] Regression check
Commands / evidence:
- `python -m pytest tests/test_network_guard_lanes.py tests/test_sandbox_block_network.py tests/ui -q` → chỉ còn 1 bài fail, fail cả trên `b7a41b3` (nhãn `ProjectRow` 'Project' chưa dịch, `tests/ui/test_i18n_khong_con_chu_cu.py`).
- `python -m pytest tests -q --ignore=tests/ui` → 4 bài fail, cả 4 cũng fail trên `b7a41b3` (`test_canonical_audit_logger`, 2 bài `test_mcp_audit_security`, `test_monitoring_tab_container`).
- Chạy cả `tests` trong một lượt thì treo ở các test dựng MainWindow trong `tests/ui`; `b7a41b3` cũng treo đúng chỗ đó.
- `check_imports.py` và `check_orphan_modules.py` PASS. `check_loc.py` báo 9 file quá dài, giống hệt trước khi sửa (không file nào do nhánh này làm dài thêm).
- Kiểm tra tay trên Windows 11: trong AppContainer, Python báo `WinError 10013`, ping/nslookup/PowerShell/curl đều không ra được mạng; cmd, git, python chạy bình thường.
- Kiểm tra tay trên Windows 11: xem trước HTML tải được 4/4 tài nguyên web khi mạng mở, 0/4 khi bật chặn; tab Graph hoạt động; tạo/huỷ trang xem trước nhiều lần không còn cảnh báo profile của Qt.
## Security Impact
Permission / credential / network / customer data impact:
- Network: khi bật công tắc, chỉ nhà cung cấp AI còn ra mạng; nội dung chat vẫn gửi tới nhà cung cấp AI.
- Permission: lần đầu chạy lệnh trong sandbox, app **thêm quyền (ACE) cho SID AppContainer** trên thư mục làm việc (ghi), thư mục cài Python gốc (đọc), gốc venv và `Scripts` (đọc). Không xoá quyền nào. Thư mục chứa PySide6 không bao giờ nhận quyền kế thừa; một quyền kế thừa sai trên venv (từ bản dev trước) được tự gỡ.
- Credential: không đổi. Khi chặn, trạng thái đăng nhập M365 được đọc thẳng từ kho token trên máy, không dựng MSAL.
## Compatibility
- [x] No breaking change
- [ ] Breaking change documented
Ghi chú: `block_network` mặc định đổi từ bật sang tắt cho cấu hình mới; máy đã lưu `true` thì giữ nguyên. Khi đang chặn, lệnh dùng công cụ cài trong thư mục người dùng (ngoài Program Files) có thể báo Access denied; thư viện trong venv của app không dùng được trong sandbox.
## Reviewer Notes
- `infrastructure/sandbox/appcontainer_process.py` gọi Win32 bằng ctypes (CreateAppContainerProfile, CreateProcessW với SECURITY_CAPABILITIES) và dùng `icacls` để cấp quyền: nên xem kỹ phần cấp quyền.
- `tests/conftest.py` thêm fixture autouse gỡ `network_guard` sau mỗi test, vì `build_context()` gắn cổng này ở mức process.
- `core/task_executors.py` đang đúng bằng trần LOC nên `_run_script` được tách sang `core/task_script.py`.
🤖 Generated with [Claude Code](https://claude.com/claude-code)
Hàng "Cài đặt" là một QPushButton tự dựng icon + chữ chứ không phải hàng của
QTreeWidget như Dashboard/Giám sát, nên khoảng cách icon-chữ do layout đặt chứ
không do delegate của cây. Đặt _NAV_ROW_GAP + 4 (=10) ở đó, cộng 10px lề trái
và hộp icon 22px, đưa chữ ra x=42 trong khi delegate đặt chữ ở x=35 — thụt phải
7px, thấy rõ bằng mắt.
Lấy khe đó từ style (PM_FocusFrameHMargin + 1) thay vì tự đặt số, đúng khe mà
delegate dùng. Kèm test hồi quy đo bằng pixel trên cửa sổ thật, ép ba hàng về
cùng icon và cùng chữ để đo bố cục chứ không đo hình dáng glyph.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Vào sub-tab Project khi đang mở một project thì biểu mẫu hiện đủ dữ liệu nhưng
nút "Sửa project" vẫn mờ, không bấm được.
Trạng thái bật/mờ của ba nút chỉ được tính trong set_project_editable, mà việc
đổi project không đi qua hàm đó — _load_current chỉ nạp lại biểu mẫu.
_sync_project_buttons có chạy khi đổi sub-tab nhưng chỉ chỉnh ẩn/hiện, nên nút
hiện ra mang theo trạng thái mờ tính từ lúc dựng cửa sổ, khi chưa project nào
được chọn.
Dồn phép tính về _sync_project_buttons — nơi duy nhất biết đủ cả ba yếu tố — và
cho nó chạy thêm khi project_selected phát tín hiệu.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Thư mục làm việc vừa là sandbox (agent chỉ đọc/ghi bên trong nó) vừa là kho
kiến thức chung của project (file ở gốc được mọi đoạn chat tự đọc). Hai project
trỏ vào cùng một thư mục là đọc lẫn dữ liệu của nhau và ghi đè lên nhau — đúng
điều docstring đầu core/projects.py nói sandbox sinh ra để ngăn, nhưng không có
chỗ nào thực thi: cả ba nơi đặt thư mục đều gán thẳng output_dir rồi lưu.
Thêm core.projects.folder_conflict: so theo workspace_dir() chứ không theo
output_dir (project chưa đặt thư mục riêng vẫn đang chiếm thư mục quản lý sẵn),
chuẩn hoá đường dẫn bằng expanduser + abspath + normcase, và coi thư mục lồng
nhau là trùng — đứng ở thư mục cha vẫn với tới được file của project con.
Chặn tại cả ba nơi ghi output_dir: nút Đổi ở màn Project, thư mục mirror cloud,
và nút chọn thư mục trong tab Cowork. Chặn lúc CHỌN chứ không lúc Lưu, vì nút
Lưu chỉ ghi tên/mô tả/chỉ dẫn — chặn ở đó sẽ khoá luôn việc đổi tên một project
lỡ đang trùng thư mục.
Dữ liệu đã trùng sẵn không bị tự sửa: có nhãn cảnh báo ngay dưới ô Thư mục làm
việc, nói rõ trùng với project nào, để người dùng tự quyết.
Luật ở module riêng (presentation/workspace/project_folder_rules.py) vì nhét
vào project_editing.py sẽ đẩy file đó lên 435 dòng, vượt trần 400 của
scripts/check_loc.py.
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Trước đây công tắc chỉ chặn tool mạng của agent; lệnh shell chỉ bị proxy
giả, còn M365, Teams, nút Test, MCP đang chạy, task script, link đính kèm
task, pip tự cài và tài nguyên web trong xem trước HTML vẫn ra mạng tự do.
- Cổng chung application/network/network_guard.py, nối vào cấu hình sống
ở Composition Root; nhà cung cấp AI (chat, danh sách model, thử model)
không đi qua cổng này.
- Lệnh shell của agent và task script chạy trong Windows AppContainer
không có quyền mạng (macOS: sandbox-exec, Linux: unshare --net);
không cô lập được thì từ chối chạy.
- Không cấp quyền kế thừa của AppContainer lên thư mục chứa PySide6:
Chromium không nạp được Qt6WebEngineCore.dll và tab Graph bị hỏng.
- Bật chặn thì dừng MCP đang chạy; tool OneDrive đồng bộ trên máy vẫn dùng.
- Mặc định tắt khi mở app lần đầu; nhãn và tooltip 3 ngôn ngữ cập nhật.
- Test: tests/test_network_guard_lanes.py (có bài AppContainer thật).
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
- Trang file:// cần LocalContentCanAccessRemoteUrls mới tải được ảnh,
CSS, JS trên web; khi bật "Chặn mạng" bộ chặn request vẫn chặn.
- Dùng một QWebEngineProfile chung thuộc QApplication: profile do view
sở hữu bị huỷ trước trang, Qt báo "Release of profile requested but
WebEnginePage still not deleted" và app văng (0xc0000409 trong
Qt6Core.dll) khi chuyển tab Graph sang Folder.
- Giữ tham chiếu Python tới bộ chặn request để nó không bị thu gom.
- Thêm CLAUDE.md hướng dẫn làm việc trong repo.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
- Trả nhãn công tắc về "Block network for agent-run commands"
(vi/ja như cũ).
- Cột mục lục bên trái tính độ rộng theo kiểu chữ của mục đang chọn
(in đậm + lề 10px), trước đây "Sandbox Security Layer" bị cắt.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Summary
Nhánh
feature/perf-ui-logic: tối ưu hiệu năng/UI, sửa lỗi workspace và điều hướng, và làm cho công tắc "Block network for agent-run commands" chặn thật mọi đường ra mạng của app, trừ nhà cung cấp AI.Chặn mạng (
b78d483,8c497cf,10b8379)application/network/network_guard.py, nối vào cấu hình đang chạy ở Composition Root: đổi công tắc trong Settings là có hiệu lực ngay.sandbox-exec, Linux dùngunshare --net.file://không tải được).0xc0000409trong Qt6Core.dll); giờ dùng một profile chung thuộc QApplication.Qt6WebEngineCore.dllvà tab Graph trắng).Các commit khác trong nhánh
b7a41b3mỗi thư mục làm việc chỉ thuộc về một project ·bbdf146bật nút Sửa project khi đã có project đang mở35f24e0,cc8d5c8,2e3e719,c699bebcanh hàng / khoảng cách thanh điều hướng2759ed9không refresh workspace khi chuyển tab Cowork ·7607f44checkpoint hiệu năng và UI8548c1echặn tool mạng của agent ·caf3b74renderer GraphRAG native trên macOS ·c00b83ckhoảng cách metadata hàng project ·a04f8a9ẩn picker workspace cloudChange Type
Related Work
Cowork Task:
Core Repo: http://34.143.229.138/gitea-admin/fsg-ai-core-assets
Core AI Issue:
Core Task:
Related PR:
Scope
What is intentionally included?
tests/test_network_guard_lanes.py(có bài chạy AppContainer thật trên Windows),tests/ui/test_html_preview_remote_images.py.What is intentionally NOT included?
tls_trust).Validation
Commands / evidence:
python -m pytest tests/test_network_guard_lanes.py tests/test_sandbox_block_network.py tests/ui -q→ chỉ còn 1 bài fail, fail cả trênb7a41b3(nhãnProjectRow'Project' chưa dịch,tests/ui/test_i18n_khong_con_chu_cu.py).python -m pytest tests -q --ignore=tests/ui→ 4 bài fail, cả 4 cũng fail trênb7a41b3(test_canonical_audit_logger, 2 bàitest_mcp_audit_security,test_monitoring_tab_container).teststrong một lượt thì treo ở các test dựng MainWindow trongtests/ui;b7a41b3cũng treo đúng chỗ đó.check_imports.pyvàcheck_orphan_modules.pyPASS.check_loc.pybáo 9 file quá dài, giống hệt trước khi sửa (không file nào do nhánh này làm dài thêm).WinError 10013, ping/nslookup/PowerShell/curl đều không ra được mạng; cmd, git, python chạy bình thường.Security Impact
Permission / credential / network / customer data impact:
Scripts(đọc). Không xoá quyền nào. Thư mục chứa PySide6 không bao giờ nhận quyền kế thừa; một quyền kế thừa sai trên venv (từ bản dev trước) được tự gỡ.Compatibility
Ghi chú:
block_networkmặc định đổi từ bật sang tắt cho cấu hình mới; máy đã lưutruethì giữ nguyên. Khi đang chặn, lệnh dùng công cụ cài trong thư mục người dùng (ngoài Program Files) có thể báo Access denied; thư viện trong venv của app không dùng được trong sandbox.Reviewer Notes
infrastructure/sandbox/appcontainer_process.pygọi Win32 bằng ctypes (CreateAppContainerProfile, CreateProcessW với SECURITY_CAPABILITIES) và dùngicaclsđể cấp quyền: nên xem kỹ phần cấp quyền.tests/conftest.pythêm fixture autouse gỡnetwork_guardsau mỗi test, vìbuild_context()gắn cổng này ở mức process.core/task_executors.pyđang đúng bằng trần LOC nên_run_scriptđược tách sangcore/task_script.py.🤖 Generated with Claude Code