Skip to content

BUG-T19: Audit Log and Session Timeline are inconsistent with tool charts in multi-process runs#107

Merged
SoundBlaster merged 16 commits intomainfrom
codex/feature/BUG-T19-audit-session-consistency-r2
Feb 25, 2026
Merged

BUG-T19: Audit Log and Session Timeline are inconsistent with tool charts in multi-process runs#107
SoundBlaster merged 16 commits intomainfrom
codex/feature/BUG-T19-audit-session-consistency-r2

Conversation

@SoundBlaster
Copy link
Owner

Description

Fixes BUG-T19 by making audit/session dashboard views consistent in multi-process runs.

Changes included:

  • Refresh AuditLogger read paths from shared JSONL audit files when file metadata changes.
  • Ensure /api/audit, /api/sessions, and exports consume the same refreshed shared audit history.
  • Add regression coverage:
    • tests/unit/webui/test_audit.py
    • tests/unit/webui/test_server.py
    • tests/integration/webui/test_e2e.py
  • Update docs and DocC pages for multi-process consistency/troubleshooting guidance.
  • Archive task artifacts:
    • SPECS/ARCHIVE/BUG-T19_Audit_Log_and_Session_Timeline_are_inconsistent_with_tool_charts_in_multi_process_runs/

Validation artifact:

  • SPECS/ARCHIVE/BUG-T19_Audit_Log_and_Session_Timeline_are_inconsistent_with_tool_charts_in_multi_process_runs/BUG-T19_Validation_Report.md

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Refactoring
  • CI/CD improvement

Quality Gates

Before submitting, ensure all quality gates pass:

make check

Or run individually:

  • make test - All tests pass with ≥90% coverage
  • make lint - No linting errors
  • make format - Code is properly formatted
  • make typecheck - Type checking passes
  • make doccheck - Documentation is synced with DocC (if docs changed)

Documentation Sync

If you modified files in docs/, ensure corresponding DocC files are also updated:

docs/ file DocC file
docs/installation.md mcpbridge-wrapper.docc/Installation.md
docs/cursor-setup.md mcpbridge-wrapper.docc/CursorSetup.md
docs/claude-setup.md mcpbridge-wrapper.docc/ClaudeCodeSetup.md
docs/codex-setup.md mcpbridge-wrapper.docc/CodexCLISetup.md
docs/troubleshooting.md mcpbridge-wrapper.docc/Troubleshooting.md
docs/architecture.md mcpbridge-wrapper.docc/Architecture.md
docs/environment-variables.md mcpbridge-wrapper.docc/EnvironmentVariables.md
README.md mcpbridge-wrapper.docc/mcpbridge-wrapper.md
  • Documentation changes are synced with DocC catalog (or N/A)

Testing

  • Added/updated tests for new functionality
  • All tests pass locally
  • Manually tested the changes

Checklist

  • Code follows the project's style guidelines
  • Self-review completed
  • Comments added for complex code
  • Documentation updated (if needed)
  • No new warnings generated
  • PR title is descriptive

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9335023d63

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

@SoundBlaster SoundBlaster merged commit 1ba822d into main Feb 25, 2026
10 checks passed
@SoundBlaster SoundBlaster deleted the codex/feature/BUG-T19-audit-session-consistency-r2 branch February 25, 2026 21:28
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.

1 participant