Skip to content

fix(cosh): support credentials encryption in settings.json#90

Merged
kongche-jbw merged 2 commits intomainfrom
fix/cosh/encrypt-keys-in-settings
Apr 7, 2026
Merged

fix(cosh): support credentials encryption in settings.json#90
kongche-jbw merged 2 commits intomainfrom
fix/cosh/encrypt-keys-in-settings

Conversation

@samchu-zsl
Copy link
Copy Markdown
Collaborator

@samchu-zsl samchu-zsl commented Apr 3, 2026

Description

对 settings.json 中的敏感配置项增加安全保护,支持与明文值的向后兼容。

  • 新增统一的凭据保护模块,写入时保护、读取时还原
  • 统一修改 MCP token storage 的密钥派生方式,兼容本次修改,支持自动迁移

同时修复 pre-commit.js 中 lint-staged 的工作目录指向错误,确保提交前顺利进行 lint/format。

建议进行 rebase 合并。

Related Issue

no-issue: Non-Disclosure Issue

Type of Change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Refactoring (no functional change)
  • Performance improvement
  • CI/CD or build changes

Scope

  • cosh (copilot-shell)
  • agent-sec-core
  • os-skills
  • agentsight
  • Multiple / Project-wide

Checklist

  • [ x I have read the Contributing Guide
  • My code follows the project's code style
  • I have added tests that prove my fix is effective or that my feature works
  • I have updated the documentation accordingly
  • For cosh: Lint passes, type check passes, and tests pass
  • For agent-sec-core (Rust): cargo clippy -- -D warnings and cargo fmt --check pass
  • For agent-sec-core (Python): Ruff format and pytest pass
  • For os-skills: Skill directory structure is valid and shell scripts pass syntax check
  • Lock files are up to date (package-lock.json / Cargo.lock)

Testing

Additional Notes

@samchu-zsl samchu-zsl added this to the cosh-v2.0.3 milestone Apr 3, 2026
@samchu-zsl samchu-zsl added the type:bug Something isn't working label Apr 3, 2026
@samchu-zsl samchu-zsl self-assigned this Apr 3, 2026
@casparant casparant added the component:cosh src/copilot-shell/ label Apr 4, 2026
@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 4, 2026

CLA assistant check
All committers have signed the CLA.

@samchu-zsl samchu-zsl force-pushed the fix/cosh/encrypt-keys-in-settings branch from a4bd2e0 to 8f84cb6 Compare April 7, 2026 01:27
@samchu-zsl samchu-zsl requested a review from kongche-jbw as a code owner April 7, 2026 01:27
@samchu-zsl samchu-zsl requested a review from casparant April 7, 2026 01:33
@kongche-jbw
Copy link
Copy Markdown
Collaborator

lgtm

@kongche-jbw kongche-jbw merged commit 6fce457 into main Apr 7, 2026
16 of 17 checks passed
@samchu-zsl samchu-zsl deleted the fix/cosh/encrypt-keys-in-settings branch April 8, 2026 01:12
@kongche-jbw kongche-jbw mentioned this pull request Apr 8, 2026
22 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component:cosh src/copilot-shell/ type:bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants