Skip to content

refactor(memo): simplify memo tool argument/scope helpers#21

Merged
Yumiue merged 1 commit intofeat-memo-scope-layeringfrom
fork-pr-354-1776496154
Apr 18, 2026
Merged

refactor(memo): simplify memo tool argument/scope helpers#21
Yumiue merged 1 commit intofeat-memo-scope-layeringfrom
fork-pr-354-1776496154

Conversation

@fennoai
Copy link
Copy Markdown

@fennoai fennoai Bot commented Apr 18, 2026

Requested by @Yumiue

Summary

  • Extracted shared memo tool helpers for argument-unmarshal errors and scope schema.
  • Moved shared scope parsing into common helper to reduce duplication.
  • Kept behavior unchanged and validated with full test suite.

Generated with [codeagent](https://github.com/qbox/codeagent)
Co-authored-by: Yumiue <188874804+Yumiue@users.noreply.github.com>
@codecov-commenter
Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 96.66667% with 1 line in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/tools/memo/list.go 50.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@Yumiue Yumiue merged commit c5fd3e1 into feat-memo-scope-layering Apr 18, 2026
1 check passed
@fennoai fennoai Bot deleted the fork-pr-354-1776496154 branch April 18, 2026 08:02
Yumiue pushed a commit that referenced this pull request Apr 24, 2026
…lution

fix(tui): resolve startup view compile break and restore tick flow
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants