Files
Yichen Jiang 9b0f6f9017 Merge remote-tracking branch 'origin/master' into worktree/web-file-session-references
# Conflicts:
#	.agents/notes/implemented/architecture/2026-07-25-web-command-surfaces-and-assembly.i18n.yaml
#	.agents/notes/implemented/architecture/2026-07-25-web-command-surfaces-and-assembly.md
#	.agents/notes/implemented/architecture/2026-07-25-web-command-surfaces-and-assembly.zh.md
#	.agents/notes/implemented/feature/2026-07-21-cross-session-references.i18n.yaml
#	.agents/notes/implemented/feature/2026-07-21-cross-session-references.md
#	.agents/notes/implemented/feature/2026-07-21-cross-session-references.zh.md
#	.agents/notes/implemented/feature/2026-07-27-web-subagent-conversations.i18n.yaml
#	apps/cli/config/web.cordis.yml
#	apps/cli/package.json
#	apps/web/tests/scaffold.ts
#	docs/capability-seams.md
#	docs/config-catalog.md
#	docs/cordis-catalog/services.md
#	docs/event-producer-consumer.md
#	docs/module-graph.md
#	packages/client/connection/src/client/api.ts
#	packages/client/connection/src/client/fixture.ts
#	packages/client/connection/src/client/index.ts
#	packages/client/runtime/src/client/contract/session.ts
#	packages/client/runtime/src/client/sessions/session.ts
#	packages/client/runtime/tests/session.client.spec.ts
#	packages/client/ui-conversation/README.i18n.yaml
#	packages/client/ui-conversation/README.md
#	packages/client/ui-conversation/README.zh.md
#	packages/client/ui-conversation/src/client/chat/ChatView.tsx
#	packages/client/ui-conversation/src/client/chat/MessageItem.tsx
#	packages/client/ui-conversation/src/client/chat/chat-flow.ts
#	packages/client/ui-conversation/src/client/input/facade.ts
#	packages/client/ui-conversation/src/client/input/hub.ts
#	packages/client/ui-conversation/tests/apply-inject.client.spec.tsx
#	packages/client/ui-conversation/tests/chat-branch-tails.client.spec.tsx
#	packages/client/ui-conversation/tests/chat-view.client.spec.tsx
#	packages/client/ui-conversation/tests/input-bar.client.spec.tsx
#	packages/client/ui-conversation/tests/input-matrix.client.spec.tsx
#	packages/client/ui-conversation/tests/input-scenarios.client.spec.tsx
#	packages/client/ui-conversation/tests/skeleton.client.spec.tsx
#	packages/client/ui-input-trigger/package.json
#	packages/client/ui-input-trigger/src/types.ts
#	packages/client/ui-jobs/README.i18n.yaml
#	packages/client/ui-slash/README.md
#	packages/client/ui-slash/README.zh.md
#	packages/client/ui-subagent/README.i18n.yaml
#	packages/client/ui-subagent/README.md
#	packages/client/ui-subagent/README.zh.md
#	packages/client/ui-subagent/package.json
#	packages/client/ui-subagent/src/client/index.ts
#	packages/client/ui-subagent/tests/browser-plugin.client.spec.ts
#	packages/client/ui-subagent/tsconfig.json
#	packages/context/session-reference/README.i18n.yaml
#	packages/context/session-reference/README.md
#	packages/context/session-reference/README.zh.md
#	packages/cordis/tool-cordis/src/api-catalog.ts
#	packages/core/session/README.i18n.yaml
#	packages/core/session/README.zh.md
#	packages/host/apiproxy/README.i18n.yaml
#	packages/host/apiproxy/README.md
#	packages/host/apiproxy/README.zh.md
#	packages/host/apiproxy/src/api-proxy.ts
#	packages/host/apiproxy/src/api/index.ts
#	packages/host/apiproxy/src/api/rpc-map.ts
#	packages/host/apiproxy/src/api/rpc.schema.ts
#	packages/host/apiproxy/src/api/rpc.ts
#	packages/host/apiproxy/src/api/sessions.ts
#	packages/host/apiproxy/src/fetch/client.ts
#	packages/host/apiproxy/src/fetch/handler.ts
#	packages/host/apiproxy/src/index.ts
#	packages/host/apiproxy/tests/client-handler.spec.ts
#	packages/host/apiproxy/tsconfig.json
#	pnpm-lock.yaml
#	scripts/gen-cordis-catalog.ts
#	scripts/gen-doc-graphs.ts
#	scripts/verify-package-readme-model-experience.ts
#	tsconfig.base.json
#	tsconfig.client.json
#	vitest.config.ts
2026-08-14 16:18:40 +08:00

127 lines
5.0 KiB
TypeScript

// detectTrigger word-boundary, position, guard-tier, and span behavior.
// URL rule pinned here: '/' is dead when its predecessor is
// another '/' (second slash of '//') or a ':' itself preceded by a
// non-whitespace char (scheme separator) — this is the concrete rule chosen
// to honor "no trigger inside URLs".
import { describe, expect, it } from 'vitest'
import { detectTrigger } from '../src/core/detect.ts'
import type { TriggerGuard } from '../src/types.ts'
const plain: TriggerGuard = { tier: 'plain' }
const claimed: TriggerGuard = { tier: 'claimed' }
const frozen: TriggerGuard = { tier: 'frozen' }
/** Hit at the end of the draft under the plain tier. */
const atEnd = (draft: string, guard: TriggerGuard = plain) => detectTrigger(draft, draft.length, guard)
describe('detectTrigger word boundaries', () => {
it('triggers at start of draft', () => {
expect(atEnd('/go')).toMatchObject({ trigger: '/', query: 'go', position: 'leading' })
expect(atEnd('@wo')).toMatchObject({ trigger: '@', query: 'wo', position: 'leading' })
})
it('triggers after whitespace, newline, and punctuation', () => {
expect(atEnd('say /co')).toMatchObject({ trigger: '/', query: 'co' })
expect(atEnd('line1\n/go')).toMatchObject({ trigger: '/', query: 'go', position: 'inline' })
expect(atEnd('see (/go')).toMatchObject({ trigger: '/', query: 'go' })
expect(atEnd('ping @wo')).toMatchObject({ trigger: '@', query: 'wo' })
})
it('does not trigger after a word character', () => {
expect(atEnd('user@host')).toBeNull()
expect(atEnd('a/b')).toBeNull()
expect(atEnd('foo_1@bar')).toBeNull()
})
it('does not trigger on URL slashes', () => {
// Both '//' slashes: first blocked by the ':' rule, second by the '/' rule.
expect(atEnd('https://example')).toBeNull()
expect(atEnd('see https://example')).toBeNull()
// Path slashes deeper in the URL sit after word chars.
expect(atEnd('https://a.b/c/d')).toBeNull()
// Single slash after a scheme-like colon (mailto:/, C:/).
expect(atEnd('C:/path')).toBeNull()
})
it('still triggers when a colon is not a scheme separator', () => {
// ':' preceded by whitespace / at index 0 is ordinary punctuation.
expect(atEnd('note: /go')).toMatchObject({ trigger: '/', query: 'go' })
expect(atEnd(':/go')).toMatchObject({ trigger: '/', query: 'go' })
})
it('stops the backward scan at whitespace', () => {
// Space after the token: no trigger at the caret anymore.
expect(atEnd('/goal x')).toBeNull()
expect(atEnd('@worker done')).toBeNull()
})
it('finds the nearest trigger left of the caret', () => {
expect(atEnd('/goal @wor')).toMatchObject({ trigger: '@', query: 'wor' })
})
})
describe('detectTrigger position', () => {
it('treats a draft whose leading trim (incl. newlines) starts at the token as leading', () => {
expect(atEnd('\n\n/goal')).toMatchObject({ position: 'leading' })
expect(atEnd(' \n /goal')).toMatchObject({ position: 'leading' })
})
it('treats a token after non-whitespace text as inline', () => {
expect(atEnd('第一行\n/goal')).toMatchObject({ position: 'inline' })
expect(atEnd('a /goal')).toMatchObject({ position: 'inline' })
})
})
describe('detectTrigger guard tiers', () => {
it('claimed suppresses "/" everywhere but keeps "@"', () => {
expect(atEnd('/co', claimed)).toBeNull()
expect(atEnd('args /path', claimed)).toBeNull()
expect(atEnd('/goal @wor', claimed)).toMatchObject({ trigger: '@', query: 'wor' })
})
it('a suppressed "/" is scanned through like an ordinary char', () => {
// '/x' right of the caret path: scan passes the dead '/' and hits nothing.
expect(detectTrigger('/goal /x', 8, claimed)).toBeNull()
})
it('frozen suppresses both triggers', () => {
expect(atEnd('/co', frozen)).toBeNull()
expect(atEnd('@wo', frozen)).toBeNull()
})
})
describe('detectTrigger span and query', () => {
it('keeps an open quoted @file token active across spaces', () => {
const draft = 'read @"docs/design notes'
expect(atEnd(draft)).toMatchObject({
trigger: '@',
query: 'docs/design notes',
quoted: true,
position: 'inline',
span: { start: 5, end: draft.length },
})
})
it('spans trigger char to caret with a placeholder draftRev', () => {
const hit = detectTrigger('say /goal', 9, plain)
expect(hit?.span).toEqual({ start: 4, end: 9, draftRev: 0 })
expect(hit?.query).toBe('goal')
})
it('cuts the query at a mid-token caret', () => {
const hit = detectTrigger('/goal', 3, plain)
expect(hit).toMatchObject({ query: 'go', span: { start: 0, end: 3 } })
})
it('returns null at caret 0 and on empty drafts', () => {
expect(detectTrigger('', 0, plain)).toBeNull()
expect(detectTrigger('/goal', 0, plain)).toBeNull()
})
it('handles multi-line drafts with the token on a later line', () => {
const draft = 'first line\nsecond /com'
const hit = detectTrigger(draft, draft.length, plain)
expect(hit).toMatchObject({ trigger: '/', query: 'com', position: 'inline', span: { start: 18, end: 22 } })
})
})