Name: [ADD] AI Cognitive Wave-1: Memory L0-L3 + Skill runtime + Pulse cost gate + KG temporal (supersedes #53)

State: Failed finished in 14m

PR State: merged

PR Author: David Tran

PR Author Email:

PR: #54

Committer: David Tran

Committer Email: davidtran.hp@gmail.com

Commit: 2715f36e899bcce65baf0d7949f824f57b523f78

Description:

                                            [FIX] viin_ai: clear Runbot errors and warnings (PR #54)

Resolve every Runbot build error and warning on the Cognitive Wave-1
branch without masking any behavior. Each fix is root-cause proven and
verified by a real Odoo 17 test run on a fresh DB (--skip-auto-install).

- viin_ai_chat: fix stale mock signature in test_user_error_passed_through.
  _user_err now matches _collect_allowed_tools(self, applied_skills=None),
  which viin_ai_skill's _resolve_run_skills override calls positionally.
  This was the only build-breaking failure.

- viin_ai_base: guard the _record_error_usage separate-cursor write so a
  provider_id/model_id not yet visible to that cursor is set NULL instead
  of raising a ForeignKeyViolation. Removes the odoo.sql_db ERROR noise
  while preserving the trace-durability invariant (the error usage.log is
  still committed independently). Add a regression test that verifies via
  a fresh cursor.

- viin_ai_approval: remove the dead advisory tool ACL-gap warning in
  _advisory_agent_has_tools (unreachable - every caller runs sudo, so it
  could never detect a real gap); this clears the false-positive log.
  Assert the H8 executable-tools refusal warning via assertLogs instead of
  letting it leak. Add mail_notify_force_send=False to
  TestAdvisoryInfra.setUpClass to stop the mail.catchall send ERROR.

- viin_ai_memory: align the promotion.log / observation owner ir.rule perms
  with ir.model.access.csv (promotion.log read-only, observation
  append-only), clearing the to_base record-rule validator warning. Assert
  the BYOK no-provider skip warning via assertLogs. Add ACL regression
  tests driven via with_user.

- viin_ai_agent: assert the sudo-escalation refusal security warning via
  assertLogs so the expected signal is captured, not leaked to the log.
                                            

Branch: 17.0

Instance ID: 0

Age:

Up-time: