Coder/deepseek-harness
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(client-connection): drop content copy from fixture web views after base merge

Browse Source 
The base (feat/web-presenter) removed the content field from
WebSearchResultView/WebFetchResultView: the web card carries no content
copy and a capability-less UI falls back to the raw tool/result content
(web-result-card note). The fixture still constructed both web result
views with a content field, failing the e2e build with TS2353. Drop the
content spread and the stale Omit key; the fixture already emits the same
text as the tool/result content the fallback path renders.
This commit is contained in:
Chinesezjc
2026-07-30 21:26:37 +08:00
parent 896a6d3cba
commit 60a3cbeb8e
1 changed files with 9 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

17
packages/client/connection/src/client/fixture.ts
View File

@@ -143,9 +143,9 @@ const TERMINAL_EXIT_STATUS: Record<string, { exitCode: number } | { signal: stri
* snippet and a date, a source with no title (its hostname labels the link) and * snippet and a date, a source with no title (its hostname labels the link) and
* a snippet but no date, and a source with a title and a date but no snippet. * a snippet but no date, and a source with a title and a date but no snippet.
* `truncated` marks the capped indicator. The shape is the contract's own * `truncated` marks the capped indicator. The shape is the contract's own
* search view minus its wire discriminants and fallback content. * search view minus its wire discriminants.
*/ */
const WEB_SEARCH_RESULT: Omit<Extract<ToolResultView, { card: 'web'; kind: 'search' }>, 'card' | 'kind' | 'content'> = { const WEB_SEARCH_RESULT: Omit<Extract<ToolResultView, { card: 'web'; kind: 'search' }>, 'card' | 'kind'> = {
answer: 'DeepSeek Harness is a plugin-based agent harness on vendored Cordis where **every capability is a plugin**.', answer: 'DeepSeek Harness is a plugin-based agent harness on vendored Cordis where **every capability is a plugin**.',
sources: [ sources: [
{ {
@@ -168,7 +168,7 @@ const WEB_SEARCH_RESULT: Omit<Extract<ToolResultView, { card: 'web'; kind: 'sear
} }
/** The `web_fetch` result view for fixture turn 67, authored inline for the same reason. */ /** The `web_fetch` result view for fixture turn 67, authored inline for the same reason. */
const WEB_FETCH_RESULT: Omit<Extract<ToolResultView, { card: 'web'; kind: 'fetch' }>, 'card' | 'kind' | 'content'> = { const WEB_FETCH_RESULT: Omit<Extract<ToolResultView, { card: 'web'; kind: 'fetch' }>, 'card' | 'kind'> = {
url: 'https://www.deepseek.com/blog/harness-architecture', url: 'https://www.deepseek.com/blog/harness-architecture',
statusCode: 200, statusCode: 200,
truncated: false, truncated: false,
@@ -402,14 +402,15 @@ function presentResult(name: string, argsRaw: string, resultText: string): ToolR
const call = presentCall(name, argsRaw) const call = presentCall(name, argsRaw)
if (call === undefined) return undefined if (call === undefined) return undefined
// The web tools keep a generic pending card, so their result card is chosen // The web tools keep a generic pending card, so their result card is chosen
// by tool name rather than by the pending card tag: the structured `web` // by tool name rather than by the pending card tag: the structured `web` card
// card the frontend consumes, with the model-facing text kept as the // the frontend consumes. The view carries no `content` copy (per the contract
// capability-less fallback content. // and the web-result-card note); a capability-less UI falls back to the raw
// `tool/result` content, which this fixture emits from `resultText`.
if (name === 'web_search') { if (name === 'web_search') {
return { card: 'web', kind: 'search', ...WEB_SEARCH_RESULT, content: text(resultText) } return { card: 'web', kind: 'search', ...WEB_SEARCH_RESULT }
} }
if (name === 'web_fetch') { if (name === 'web_fetch') {
return { card: 'web', kind: 'fetch', ...WEB_FETCH_RESULT, content: text(resultText) } return { card: 'web', kind: 'fetch', ...WEB_FETCH_RESULT }
} }
switch (call.card) { switch (call.card) {
case 'terminal': case 'terminal':