test(context-injector): remove keyword-specific test cases
Keyword detection is now handled by claude-code-hooks, not by context-injector messages.transform hook. Remove tests for keyword injection that are no longer applicable. 🤖 GENERATED WITH ASSISTANCE OF [OhMyOpenCode](https://github.com/code-yeongyu/oh-my-opencode)
This commit is contained in:
@@ -234,7 +234,7 @@ describe("createContextInjectorMessagesTransformHook", () => {
|
||||
expect(output.messages[3].parts[0].text).toBe("Second message")
|
||||
})
|
||||
|
||||
it("does nothing when no pending context and no keywords", async () => {
|
||||
it("does nothing when no pending context", async () => {
|
||||
// #given
|
||||
const hook = createContextInjectorMessagesTransformHook(collector)
|
||||
const sessionID = "ses_transform2"
|
||||
@@ -249,63 +249,6 @@ describe("createContextInjectorMessagesTransformHook", () => {
|
||||
expect(output.messages.length).toBe(1)
|
||||
})
|
||||
|
||||
it("injects synthetic message when user message contains ulw keyword", async () => {
|
||||
// #given
|
||||
const hook = createContextInjectorMessagesTransformHook(collector)
|
||||
const sessionID = "ses_transform_keyword"
|
||||
const messages = [createMockMessage("user", "ulw do this task", sessionID)]
|
||||
// eslint-disable-next-line @typescript-eslint/no-explicit-any
|
||||
const output = { messages } as any
|
||||
|
||||
// #when
|
||||
await hook["experimental.chat.messages.transform"]!({}, output)
|
||||
|
||||
// #then
|
||||
expect(output.messages.length).toBe(2)
|
||||
expect(output.messages[0].parts[0].synthetic).toBe(true)
|
||||
expect(output.messages[0].parts[0].text).toContain("ultrawork")
|
||||
expect(output.messages[1].parts[0].text).toBe("ulw do this task")
|
||||
})
|
||||
|
||||
it("injects synthetic message when user message contains ultrawork keyword", async () => {
|
||||
// #given
|
||||
const hook = createContextInjectorMessagesTransformHook(collector)
|
||||
const sessionID = "ses_transform_ultrawork"
|
||||
const messages = [createMockMessage("user", "ultrawork please", sessionID)]
|
||||
// eslint-disable-next-line @typescript-eslint/no-explicit-any
|
||||
const output = { messages } as any
|
||||
|
||||
// #when
|
||||
await hook["experimental.chat.messages.transform"]!({}, output)
|
||||
|
||||
// #then
|
||||
expect(output.messages.length).toBe(2)
|
||||
expect(output.messages[0].parts[0].synthetic).toBe(true)
|
||||
})
|
||||
|
||||
it("combines keyword context with collector context", async () => {
|
||||
// #given
|
||||
const hook = createContextInjectorMessagesTransformHook(collector)
|
||||
const sessionID = "ses_transform_combined"
|
||||
collector.register(sessionID, {
|
||||
id: "extra",
|
||||
source: "custom",
|
||||
content: "Extra context from collector",
|
||||
})
|
||||
const messages = [createMockMessage("user", "ulw combined test", sessionID)]
|
||||
// eslint-disable-next-line @typescript-eslint/no-explicit-any
|
||||
const output = { messages } as any
|
||||
|
||||
// #when
|
||||
await hook["experimental.chat.messages.transform"]!({}, output)
|
||||
|
||||
// #then
|
||||
expect(output.messages.length).toBe(2)
|
||||
const syntheticText = output.messages[0].parts[0].text
|
||||
expect(syntheticText).toContain("ultrawork")
|
||||
expect(syntheticText).toContain("Extra context from collector")
|
||||
})
|
||||
|
||||
it("does nothing when no user messages", async () => {
|
||||
// #given
|
||||
const hook = createContextInjectorMessagesTransformHook(collector)
|
||||
|
||||
Reference in New Issue
Block a user