Merge branch 'codex/simp-unify-agent-session-id' into codex/simp-ui-identity-residue

This commit is contained in:
Tianyi Cui
2026-07-16 00:51:33 +08:00
4 changed files with 25 additions and 12 deletions

View File

@@ -257,6 +257,7 @@ describe('CreateWizard and scaffolder', () => {
expect(index).toContain('SdkBootContext') expect(index).toContain('SdkBootContext')
expect(index).toContain('ctx.agents.create') expect(index).toContain('ctx.agents.create')
expect(index).toContain('agentOptions: { model: "deepseek-v4-flash" }') expect(index).toContain('agentOptions: { model: "deepseek-v4-flash" }')
expect(index).not.toContain('AgentId')
const tsconfig = parseGeneratedTsConfig(await readFile(join(target, 'tsconfig.base.json'), 'utf8')) const tsconfig = parseGeneratedTsConfig(await readFile(join(target, 'tsconfig.base.json'), 'utf8'))
const manifest = parseGeneratedPackageManifest(await readFile(join(target, 'package.json'), 'utf8')) const manifest = parseGeneratedPackageManifest(await readFile(join(target, 'package.json'), 'utf8'))
expect(tsconfig.compilerOptions.types).toEqual(['node']) expect(tsconfig.compilerOptions.types).toEqual(['node'])

View File

@@ -4,6 +4,7 @@
* @module @deepseek-ai/dsh-helper/features/builtin/app * @module @deepseek-ai/dsh-helper/features/builtin/app
*/ */
import { JsExpression } from '../../documents/cordis-yaml-file.ts'
import { featureId } from '../../ids.ts' import { featureId } from '../../ids.ts'
import type { ProjectProfile } from '../../project/types.ts' import type { ProjectProfile } from '../../project/types.ts'
import { import {
@@ -94,11 +95,11 @@ class AppOption extends FeatureOption {
name: '@deepseek-ai/dsh-stdio', name: '@deepseek-ai/dsh-stdio',
config: { config: {
welcome: 'agent REPL ready. Give it a coding task.', welcome: 'agent REPL ready. Give it a coding task.',
agent: 'main', sessionId: new JsExpression('process.env.DSH_SDK_SESSION_ID'),
}, },
}, ['welcome', 'agent'], config => [ }, ['welcome', 'sessionId'], config => [
...optionalString(config, 'welcome'), ...optionalString(config, 'welcome'),
...requiredString(config, 'agent'), ...config.sessionId instanceof JsExpression ? [] : requiredString(config, 'sessionId'),
]), ]),
]) ])
case 'embed': case 'embed':

View File

@@ -2,14 +2,12 @@
import { startSDK, type SdkBootContext } from '@deepseek-ai/dsh-scripts' import { startSDK, type SdkBootContext } from '@deepseek-ai/dsh-scripts'
{{else}} {{else}}
import { randomUUID } from 'node:crypto' import { randomUUID } from 'node:crypto'
import { AgentId } from '@deepseek-ai/dsh-agent'
import { SessionId } from '@deepseek-ai/dsh-session' import { SessionId } from '@deepseek-ai/dsh-session'
import { startSDK, type SdkBootContext } from '@deepseek-ai/dsh-scripts' import { startSDK, type SdkBootContext } from '@deepseek-ai/dsh-scripts'
{{/if}} {{/if}}
/** Boot this project's cordis.yml when invoked by dsh-scripts. */ /** Boot this project's cordis.yml when invoked by dsh-scripts. */
export async function main(boot: SdkBootContext) { export async function main(boot: SdkBootContext) {
const ctx = await startSDK(new URL('./cordis.yml', import.meta.url))
{{#if isStdio}} {{#if isStdio}}
const model = boot.args.model const model = boot.args.model
if (typeof model !== 'string' || model.length === 0) throw new Error('stdio startup requires --model=<name>') if (typeof model !== 'string' || model.length === 0) throw new Error('stdio startup requires --model=<name>')
@@ -17,24 +15,26 @@ export async function main(boot: SdkBootContext) {
if (resume !== undefined && (typeof resume !== 'string' || resume.length === 0)) { if (resume !== undefined && (typeof resume !== 'string' || resume.length === 0)) {
throw new Error('stdio startup requires --resume=<session-id>') throw new Error('stdio startup requires --resume=<session-id>')
} }
const sessionId = SessionId(resume ?? `main-session-${randomUUID()}`)
process.env.DSH_SDK_SESSION_ID = sessionId
{{/if}}
const ctx = await startSDK(new URL('./cordis.yml', import.meta.url))
{{#if isStdio}}
if (resume === undefined) { if (resume === undefined) {
await ctx.agents.create({ await ctx.agents.create({
agentId: AgentId('main'), sessionId,
sessionId: SessionId(`main-session-${randomUUID()}`),
meta: { cwd: boot.cwd }, meta: { cwd: boot.cwd },
agentOptions: { model }, agentOptions: { model },
}) })
} else { } else {
await ctx.agents.resume({ await ctx.agents.resume({
agentId: AgentId('main'), resumeSessionId: sessionId,
resumeSessionId: SessionId(resume),
agentOptions: { model }, agentOptions: { model },
}) })
} }
{{else}} {{else}}
{{#if isEmbed}} {{#if isEmbed}}
await ctx.agents.create({ await ctx.agents.create({
agentId: AgentId('main'),
sessionId: SessionId(`main-session-${randomUUID()}`), sessionId: SessionId(`main-session-${randomUUID()}`),
meta: { cwd: boot.cwd }, meta: { cwd: boot.cwd },
agentOptions: { model: {{modelLiteral}} }, agentOptions: { model: {{modelLiteral}} },

View File

@@ -167,6 +167,10 @@ describe('SdkProject and ProjectEditSession', () => {
expect(index).toContain('SdkBootContext') expect(index).toContain('SdkBootContext')
expect(index).toContain('agents.create') expect(index).toContain('agents.create')
expect(index).toContain('boot.args.resume') expect(index).toContain('boot.args.resume')
expect(index).not.toContain('AgentId')
expect(index).toContain('const sessionId = SessionId(resume ?? `main-session-${randomUUID()}`)')
expect(index).toContain('process.env.DSH_SDK_SESSION_ID = sessionId')
expect(index).toContain('resumeSessionId: sessionId')
expect(project.packageManifest().scripts).toEqual({ expect(project.packageManifest().scripts).toEqual({
dev: 'dsh-sdk dev index.ts -- --model="deepseek-v4-flash"', dev: 'dsh-sdk dev index.ts -- --model="deepseek-v4-flash"',
build: 'dsh-sdk build', build: 'dsh-sdk build',
@@ -175,7 +179,11 @@ describe('SdkProject and ProjectEditSession', () => {
config: 'dsh-sdk config', config: 'dsh-sdk config',
}) })
expect(await readFile(join(project.root, '.env.example'), 'utf8')).toContain('EXA_API_KEY=') expect(await readFile(join(project.root, '.env.example'), 'utf8')).toContain('EXA_API_KEY=')
expect(project.cordis.entry('stdio')?.config).toMatchObject({ agent: 'main' }) expect(project.cordis.entry('stdio')?.config?.sessionId).toMatchObject({
source: 'process.env.DSH_SDK_SESSION_ID',
})
expect(await readFile(join(project.root, 'cordis.yml'), 'utf8'))
.toContain('sessionId: !!js process.env.DSH_SDK_SESSION_ID')
expect(project.cordis.entry('stdio')?.config).not.toHaveProperty('model') expect(project.cordis.entry('stdio')?.config).not.toHaveProperty('model')
expect(project.cordis.entry('agent-loop')?.config).toEqual({ agents: [] }) expect(project.cordis.entry('agent-loop')?.config).toEqual({ agents: [] })
expect(project.cordis.entry('system-prompt')?.config?.persona).toContain('{{cwd}}') expect(project.cordis.entry('system-prompt')?.config?.persona).toContain('{{cwd}}')
@@ -286,7 +294,10 @@ describe('SdkProject and ProjectEditSession', () => {
const embed = (await embedEdit.commit()).project const embed = (await embedEdit.commit()).project
expect(embed.profile.runInterface).toBe('embed') expect(embed.profile.runInterface).toBe('embed')
expect(await readFile(join(embed.root, 'README.md'), 'utf8')).toContain('Embed the harness') expect(await readFile(join(embed.root, 'README.md'), 'utf8')).toContain('Embed the harness')
expect(await readFile(join(embed.root, 'index.ts'), 'utf8')).toContain('agents.create') const embedIndex = await readFile(join(embed.root, 'index.ts'), 'utf8')
expect(embedIndex).toContain('agents.create')
expect(embedIndex).toContain("import { SessionId } from '@deepseek-ai/dsh-session'")
expect(embedIndex).not.toContain('AgentId')
await writeFile(join(embed.root, 'README.md'), '# Custom README\n') await writeFile(join(embed.root, 'README.md'), '# Custom README\n')
const modified = await SdkProject.open(embed.root) const modified = await SdkProject.open(embed.root)