forked from lance-format/lance-graph
-
Notifications
You must be signed in to change notification settings - Fork 0
callcenter: D-ODOO-SAV-4 odoo-savant reasoners + audit/Policy fixes #420
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
Merged
Merged
Changes from all commits
Commits
Show all changes
10 commits
Select commit
Hold shift + click to select a range
ea2a378
fix(callcenter): with_jsonl_audit returns Result<Self, AuditError>
claude f3d62f5
docs(tech-debt): document AriGraph episodic-fidelity options + JSON s…
claude dbd4fed
docs(plans): multi-server-cognition-expansion-v1 — kill the legacy-st…
claude db021aa
docs(board): record ea2a378 with_jsonl_audit fix — EPIPHANIES E-AUDIT…
claude b526d9c
feat(callcenter): re-export rbac Policy + smb_policy through the facade
claude 20da477
Merge remote-tracking branch 'origin/main' into claude/activate-lance…
claude 26ba873
feat(callcenter): D-ODOO-SAV-4 — odoo-savant Reasoner layer (4 impls …
claude 5358fe6
chore(.claude): enforce Read over grep/sed/head/tail (hook + warden +…
claude 87ab547
Revert "chore(.claude): enforce Read over grep/sed/head/tail (hook + …
claude 4162689
fix(callcenter savant_reasoners): add missing SavantSuggestion payloa…
claude 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
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.
Status claim inconsistent with PR context and file governance rules.
Line 40 states "not yet a PR" but this entry is being added as part of PR
#420(per PR objectives). Additionally, the file header (lines 3-6) states this tracks "Every merged PR", yet this entry is being prepended pre-merge.This violates the historical-record contract:
#420contextRecommendation: Either update line 40 to reference PR
#420explicitly, or defer adding this entry until after PR#420merges. The append-only discipline works best when entries are truly immutable historical facts, not in-flight documentation.Suggested fix
Alternatively, consider moving this entire entry addition to a post-merge housekeeping commit so it becomes a true historical record per the file's stated purpose.
🤖 Prompt for AI Agents