Skip to content

docs(filters): refresh on_invalid type rendering #613

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.

Sign up for GitHub

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

Open
aslakknutsen wants to merge 1 commit into
base: main
Choose a base branch
from
+4 −4
Open

docs(filters): refresh on_invalid type rendering #613

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion docs/filters/a2a.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ When `task_routing.enabled` is true, the filter captures task and context owners
| `headers.version` | string | no | Header name for A2A version (e.g. `x-praxis-a2a-version`). |
| `max_body_bytes` | integer | no | Maximum body size in bytes for `StreamBuffer`. |
| `method_aliases` | object<string, string> | no | Method aliases for compatibility (slash-delimited → `PascalCase`). |
| `on_invalid` | `continue` \| `reject` \| `error` | no | Invalid input handling behavior. |
| `on_invalid` | OnInvalidBehavior | no | Invalid input handling behavior. |
| `task_routing` | TaskRoutingConfig | no | Task-ownership routing configuration. |
| `task_routing.enabled` | bool | no | Whether task routing is enabled. |
| `task_routing.max_response_body_bytes` | integer | no | Maximum response body bytes to buffer for task route capture. |
Expand Down
2 changes: 1 addition & 1 deletion docs/filters/anthropic_messages_format.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ Classifies Anthropic Messages API requests and promotes routing facts to headers

| Field | Type | Required | Description |
|-------|------|---------|-------------|
| `on_invalid` | `continue` \| `reject` \| `error` | no | Behavior when the body cannot be classified. |
| `on_invalid` | OnInvalidBehavior | no | Behavior when the body cannot be classified. |
| `max_body_bytes` | integer | no | Maximum body size in bytes for `StreamBuffer` mode. |
| `headers` | AnthropicMessagesFormatHeaders | no | Header names for promoted classification facts. |
| `headers.format` | string | no | Header name for the detected format. |
Expand Down
2 changes: 1 addition & 1 deletion docs/filters/mcp.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,7 @@ Supports two protocol profiles: `current` (session-based, default) and `stateles
| `headers.protocol_version` | string | no | Header name for the MCP protocol version (e.g. `x-praxis-mcp-protocol-version`). |
| `headers.session_present` | string | no | Header name for MCP session presence (e.g. `x-praxis-mcp-session-present`). |
| `max_body_bytes` | integer | no | Maximum body size in bytes for `StreamBuffer`. |
| `on_invalid` | `continue` \| `reject` \| `error` | no | Invalid input handling behavior. |
| `on_invalid` | OnInvalidBehavior | no | Invalid input handling behavior. |

@praxis-bot praxis-bot Jul 30, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

[Medium] OnInvalidBehavior is an opaque Rust type name that tells users nothing about the accepted YAML values. The previous rendering (`continue` \| `reject` \| `error`) was immediately useful. The inconsistency is visible in this very table: the next line renders cache_scope as `public` \| `private`, and two rows above header_validation.mismatch renders as `reject` \| `ignore`.

The fix belongs in the xtask, not in the generated docs. parse_shared_config_items parses praxis/filter/src/builtins/http/payload_processing/ and should pick up on_invalid.rs where the enum lives. If the ai repo's praxis-proxy-filter dependency is pinned to a version that predates that file, updating the dependency and regenerating should restore the inline variants.

Same issue in a2a.md (line 26), anthropic_messages_format.md (line 12), and openai_responses_format.md (line 22).

| `cache_scope` | `public` \| `private` | no | Cache scope for stateless responses. Requires `protocol_profile: stateless`. |
| `cache_ttl_ms` | integer | no | Cache TTL in milliseconds for stateless responses. Requires `protocol_profile: stateless`. |
| `default_version` | string | no | Fallback MCP protocol version. When omitted, derived from the profile. |
Expand Down
2 changes: 1 addition & 1 deletion docs/filters/openai_responses_format.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ Use with branch chains to route stateful and stateless requests to different clu

| Field | Type | Required | Description |
|-------|------|---------|-------------|
| `on_invalid` | `continue` \| `reject` \| `error` | no | Behavior when the body cannot be classified. |
| `on_invalid` | OnInvalidBehavior | no | Behavior when the body cannot be classified. |
| `max_body_bytes` | integer | no | Maximum body size in bytes for `StreamBuffer` mode. |
| `headers` | ResponsesFormatHeaders | no | Header names for promoted classification facts. |
| `headers.format` | string | no | Header name for the detected format (e.g. `openai_responses`, `openai_chat_completions`). |
Expand Down
Loading