Skip to content

Commit bd04e1d

Browse files
committed
docs: fix OpenAI strict-schema and Bedrock note in office-assistant MVP
Addresses PR review feedback: - ingest-meeting: OpenAI structured output is sent with strict:true, which requires every declared property to appear in `required`. The action_items item schema listed description/owner but only required title, so the summarize step would 400. Make them required and nullable (type: [string, null]) so absent values are expressible. - docs: the Bedrock swap needs `provider: bedrock` (+ region); provider defaults to openai, so changing only credential/model still sends an OpenAI-format request. Corrected the guidance. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WnKgCRfMvFcZAkoVbr8k79
1 parent 6d0413e commit bd04e1d

2 files changed

Lines changed: 19 additions & 3 deletions

File tree

packages/site/src/content/docs/office-assistant-mvp.md

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,11 +21,17 @@ Create these credentials (`mantle secrets create`) and the KB schema:
2121

2222
| Credential name | Type | Used for |
2323
| --------------- | ---- | -------- |
24-
| `openai` | `openai` | `ai/completion` (summarize + answer). Swap the `credential:` and `model:` for `bedrock` to use Claude via Bedrock. |
24+
| `openai` | `openai` | `ai/completion` (summarize + answer) |
2525
| `kb-db` | `postgres` | The knowledge-base database |
2626
| `jira` | `basic` | `jira/create_issue` (email + API token) |
2727
| `teams` | `generic` | `teams/send_message` (incoming-webhook URL) |
2828

29+
> **Using Claude via Bedrock instead of OpenAI:** in both `ai/completion` steps,
30+
> add `provider: bedrock` and a `region:` to `params`, point `credential:` at an
31+
> `aws` credential, and set `model:` to a Bedrock model ID. `provider` defaults
32+
> to `openai`, so changing only `credential`/`model` still sends an
33+
> OpenAI-format request.
34+
2935
Apply the KB schema (pure Postgres full-text search, no extensions) from
3036
[`office-assistant-kb-schema.sql`](https://github.com/dvflw/mantle/blob/main/packages/site/src/content/examples/office-assistant-kb-schema.sql):
3137

packages/site/src/content/examples/office-assistant-ingest-meeting.yaml

Lines changed: 12 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -54,12 +54,22 @@ steps:
5454
properties:
5555
title:
5656
type: string
57+
# description/owner are nullable rather than optional: OpenAI
58+
# strict structured output requires every declared property to
59+
# appear in `required`, so absent values must be expressible as
60+
# null instead of being omitted from the schema.
5761
description:
58-
type: string
62+
type:
63+
- string
64+
- "null"
5965
owner:
60-
type: string
66+
type:
67+
- string
68+
- "null"
6169
required:
6270
- title
71+
- description
72+
- owner
6373
additionalProperties: false
6474
action_items_markdown:
6575
type: string

0 commit comments

Comments
 (0)