Skip to content

fix(server): proxy namespace policy in tool context#2324

Merged
qin-ctx merged 1 commit into
mainfrom
codex/tool-context-namespace-policy
May 31, 2026
Merged

fix(server): proxy namespace policy in tool context#2324
qin-ctx merged 1 commit into
mainfrom
codex/tool-context-namespace-policy

Conversation

@Mijamind719
Copy link
Copy Markdown
Collaborator

@Mijamind719 Mijamind719 commented May 30, 2026

Summary

  • add a ToolContext.namespace_policy proxy to its wrapped RequestContext
  • cover the proxy behavior in identity tests

Related Issue

Tests

  • /Users/quemingjian/Source/OpenViking/.venv/bin/python -m ruff check openviking/server/identity.py tests/server/test_identity.py
  • /Users/quemingjian/Source/OpenViking/.venv/bin/python -m pytest -q tests/server/test_identity.py

@github-actions
Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🏅 Score: 100
🧪 PR contains tests
🔒 No security concerns identified
✅ No TODO sections
🔀 No multiple PR themes
⚡ No major issues detected

@github-actions
Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

@Mijamind719 Mijamind719 requested a review from qin-ctx May 31, 2026 07:10
@qin-ctx qin-ctx merged commit 0554011 into main May 31, 2026
5 checks passed
@qin-ctx qin-ctx deleted the codex/tool-context-namespace-policy branch May 31, 2026 10:48
@github-project-automation github-project-automation Bot moved this from Backlog to Done in OpenViking project May 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants