-
Notifications
You must be signed in to change notification settings - Fork 448
Feat/eng 1753 #306
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Closed
Closed
Feat/eng 1753 #306
Changes from 40 commits
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
4a89fd7
feat: add experience system (Phase 1–2) to capture curation signals
ngduyanhece dd55a41
fix: harden experience system Phase 3 scoring correctness and persist…
ngduyanhece 2e600c1
feat: add Phase 4 experience consolidation service
ngduyanhece 2df4a0b
fix: rename IConsolidationLlm.generate systemPrompt param to instruct…
ngduyanhece d88c47d
fix: prune static queue map entry after queue drains
ngduyanhece 3a4434b
fix: correct consolidation semantics and OOD detection
ngduyanhece 8df9da9
fix: harden experience pipeline against partial failures and edge cases
ngduyanhece ffadfbd
Merge pull request #796 from campfirein/feat/ENG-1537
ngduyanhece c4b90a0
feat: add autocontext-inspired context improvement patterns
ngduyanhece a832351
fix: resolve review issues in compression evaluator, progress contrib…
ngduyanhece 18db3d1
fix: detach session progress tracker on dispose and document naive br…
ngduyanhece c41cc02
fix: prevent compressionCount double-increment and correct attach() doc
ngduyanhece a9a0915
fix: salvage valid strings from mixed LLM arrays in bullet parser
ngduyanhece 532248d
Merge remote-tracking branch 'origin/proj/experence-domain' into feat…
ngduyanhece 9fd3f2b
Merge pull request #802 from campfirein/feat/ENG-1537
ngduyanhece 10aa128
feat: auto-export accumulated experience into agent SKILL.md files
ngduyanhece 99016f1
feat: auto-export accumulated experience into agent SKILL.md files
ngduyanhece 6a1dcc2
refactor: use options object for ExperienceHookService constructor an…
ngduyanhece 0c1daff
fix: propagate non-ENOENT errors and isolate skill export queue
ngduyanhece 7109007
Merge remote-tracking branch 'origin/proj/experence-domain' into feat…
ngduyanhece 2a96719
feat: add brv connectors sync command and remove brv-export-skill MCP…
ngduyanhece e6633cf
fix: tighten sync command, coordinator return type, and hook fail-open
ngduyanhece 17699ff
Merge pull request #803 from campfirein/feat/ENG-1556
ngduyanhece bb3ed2a
refactor: extract SkillExportResult and SkillBuildAndSyncResult into …
ngduyanhece 071b249
refactor: use SkillExportScope in SkillExportTarget and re-export it
ngduyanhece 7bcfbdc
refactor: import skill export result from shared types
ngduyanhece 33835cc
Merge remote-tracking branch 'origin/proj/experence-domain' into feat…
ngduyanhece f5bc923
Merge pull request #809 from campfirein/feat/ENG-1556
ngduyanhece f6606ca
Merge proj/ov-pattern into proj/experence-domain with conflict resolu…
ngduyanhece 1bb4a4f
feat: refactor experience domain to entry-based storage with native p…
ngduyanhece 9f4353e
Fix synthesis reflection overwrite path
ngduyanhece 19dfcb5
Document daily synthesis retention
ngduyanhece 156e569
Resolve remaining experience review comments
ngduyanhece ed16ee8
Address final PR review blockers
ngduyanhece ec018b1
chore: remove migration code, connectors sync, and add E2E tests
ngduyanhece efdc022
Resolve remaining PR review blockers
ngduyanhece 345f22a
Merge pull request #894 from campfirein/feat/ENG-1746
ngduyanhece ff7f4f1
feat: implement memory-performance correlation (Phase 2)
ngduyanhece a74d0f0
Address PR review feedback on search correlation
ngduyanhece dbbcee1
Address remaining PR review threads
ngduyanhece 3aa4745
fix: address PR review comments
ngduyanhece 3a58d5f
feat: [ENG-1917] add Star History chart section to README.
leehpham 52a2a67
Merge pull request #309 from campfirein/feat/ENG-1917
leehpham 6598f63
feat: [ENG-1910] add ByteRover Cloud section to README.
leehpham c6dbfa9
Merge pull request #310 from campfirein/feat/ENG-1910
leehpham 660a115
Merge remote-tracking branch 'origin/main' into feat/ENG-1753
ngduyanhece 5a137ea
fix: address PR review round 2 — import style, score boundary, commen…
ngduyanhece eeee720
fix: address latest PR review feedback
ngduyanhece File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nitpick (import style): Same inline dynamic
import(...)type pattern that was flagged and fixed intools-sdk.tsin the previous round — now it's reintroduced here. Consider lifting it to a top-level import to keep the file consistent with the rest of the imports:And add at the top of the file alongside other
import typedeclarations: