| 123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228 |
- import { expect, test, describe } from "bun:test"
- import { ConfigMarkdown } from "../../src/config/markdown"
- describe("ConfigMarkdown: normal template", () => {
- const template = `This is a @valid/path/to/a/file and it should also match at
- the beginning of a line:
- @another-valid/path/to/a/file
- but this is not:
- - Adds a "Co-authored-by:" footer which clarifies which AI agent
- helped create this commit, using an appropriate \`noreply@...\`
- or \`[email protected]\` email address.
- We also need to deal with files followed by @commas, ones
- with @file-extensions.md, even @multiple.extensions.bak,
- hidden directories like @.config/ or files like @.bashrc
- and ones at the end of a sentence like @foo.md.
- Also shouldn't forget @/absolute/paths.txt with and @/without/extensions,
- as well as @~/home-files and @~/paths/under/home.txt.
- If the reference is \`@quoted/in/backticks\` then it shouldn't match at all.`
- const matches = ConfigMarkdown.files(template)
- test("should extract exactly 12 file references", () => {
- expect(matches.length).toBe(12)
- })
- test("should extract valid/path/to/a/file", () => {
- expect(matches[0][1]).toBe("valid/path/to/a/file")
- })
- test("should extract another-valid/path/to/a/file", () => {
- expect(matches[1][1]).toBe("another-valid/path/to/a/file")
- })
- test("should extract paths ignoring comma after", () => {
- expect(matches[2][1]).toBe("commas")
- })
- test("should extract a path with a file extension and comma after", () => {
- expect(matches[3][1]).toBe("file-extensions.md")
- })
- test("should extract a path with multiple dots and comma after", () => {
- expect(matches[4][1]).toBe("multiple.extensions.bak")
- })
- test("should extract hidden directory", () => {
- expect(matches[5][1]).toBe(".config/")
- })
- test("should extract hidden file", () => {
- expect(matches[6][1]).toBe(".bashrc")
- })
- test("should extract a file ignoring period at end of sentence", () => {
- expect(matches[7][1]).toBe("foo.md")
- })
- test("should extract an absolute path with an extension", () => {
- expect(matches[8][1]).toBe("/absolute/paths.txt")
- })
- test("should extract an absolute path without an extension", () => {
- expect(matches[9][1]).toBe("/without/extensions")
- })
- test("should extract an absolute path in home directory", () => {
- expect(matches[10][1]).toBe("~/home-files")
- })
- test("should extract an absolute path under home directory", () => {
- expect(matches[11][1]).toBe("~/paths/under/home.txt")
- })
- test("should not match when preceded by backtick", () => {
- const backtickTest = "This `@should/not/match` should be ignored"
- const backtickMatches = ConfigMarkdown.files(backtickTest)
- expect(backtickMatches.length).toBe(0)
- })
- test("should not match email addresses", () => {
- const emailTest = "Contact [email protected] for help"
- const emailMatches = ConfigMarkdown.files(emailTest)
- expect(emailMatches.length).toBe(0)
- })
- })
- describe("ConfigMarkdown: frontmatter parsing", async () => {
- const parsed = await ConfigMarkdown.parse(import.meta.dir + "/fixtures/frontmatter.md")
- test("should parse without throwing", () => {
- expect(parsed).toBeDefined()
- expect(parsed.data).toBeDefined()
- expect(parsed.content).toBeDefined()
- })
- test("should extract description field", () => {
- expect(parsed.data.description).toBe("This is a description wrapped in quotes")
- })
- test("should extract occupation field with colon in value", () => {
- expect(parsed.data.occupation).toBe("This man has the following occupation: Software Engineer")
- })
- test("should extract title field with single quotes", () => {
- expect(parsed.data.title).toBe("Hello World")
- })
- test("should extract name field with embedded quotes", () => {
- expect(parsed.data.name).toBe('John "Doe"')
- })
- test("should extract family field with embedded single quotes", () => {
- expect(parsed.data.family).toBe("He has no 'family'")
- })
- test("should extract multiline summary field", () => {
- expect(parsed.data.summary).toBe("This is a summary\n")
- })
- test("should not include commented fields in data", () => {
- expect(parsed.data.field).toBeUndefined()
- })
- test("should extract URL with port", () => {
- expect(parsed.data.url).toBe("https://example.com:8080/path?query=value")
- })
- test("should extract time with colons", () => {
- expect(parsed.data.time).toBe("The time is 12:30:00 PM")
- })
- test("should extract value with multiple colons", () => {
- expect(parsed.data.nested).toBe("First: Second: Third: Fourth")
- })
- test("should preserve already double-quoted values with colons", () => {
- expect(parsed.data.quoted_colon).toBe("Already quoted: no change needed")
- })
- test("should preserve already single-quoted values with colons", () => {
- expect(parsed.data.single_quoted_colon).toBe("Single quoted: also fine")
- })
- test("should extract value with quotes and colons mixed", () => {
- expect(parsed.data.mixed).toBe('He said "hello: world" and then left')
- })
- test("should handle empty values", () => {
- expect(parsed.data.empty).toBeNull()
- })
- test("should handle dollar sign replacement patterns literally", () => {
- expect(parsed.data.dollar).toBe("Use $' and $& for special patterns")
- })
- test("should not parse fake yaml from content", () => {
- expect(parsed.data.fake_field).toBeUndefined()
- expect(parsed.data.another).toBeUndefined()
- })
- test("should extract content after frontmatter without modification", () => {
- expect(parsed.content).toContain("Content that should not be parsed:")
- expect(parsed.content).toContain("fake_field: this is not yaml")
- expect(parsed.content).toContain("url: https://should-not-be-parsed.com:3000")
- })
- })
- describe("ConfigMarkdown: frontmatter parsing w/ empty frontmatter", async () => {
- const result = await ConfigMarkdown.parse(import.meta.dir + "/fixtures/empty-frontmatter.md")
- test("should parse without throwing", () => {
- expect(result).toBeDefined()
- expect(result.data).toEqual({})
- expect(result.content.trim()).toBe("Content")
- })
- })
- describe("ConfigMarkdown: frontmatter parsing w/ no frontmatter", async () => {
- const result = await ConfigMarkdown.parse(import.meta.dir + "/fixtures/no-frontmatter.md")
- test("should parse without throwing", () => {
- expect(result).toBeDefined()
- expect(result.data).toEqual({})
- expect(result.content.trim()).toBe("Content")
- })
- })
- describe("ConfigMarkdown: frontmatter parsing w/ Markdown header", async () => {
- const result = await ConfigMarkdown.parse(import.meta.dir + "/fixtures/markdown-header.md")
- test("should parse and match", () => {
- expect(result).toBeDefined()
- expect(result.data).toEqual({})
- expect(result.content.trim()).toBe(`# Response Formatting Requirements
- Always structure your responses using clear markdown formatting:
- - By default don't put information into tables for questions (but do put information into tables when creating or updating files)
- - Use headings (##, ###) to organise sections, always
- - Use bullet points or numbered lists for multiple items
- - Use code blocks with language tags for any code
- - Use **bold** for key terms and emphasis
- - Use tables when comparing options or listing structured data
- - Break long responses into logical sections with headings`)
- })
- })
- describe("ConfigMarkdown: frontmatter has weird model id", async () => {
- const result = await ConfigMarkdown.parse(import.meta.dir + "/fixtures/weird-model-id.md")
- test("should parse and match", () => {
- expect(result).toBeDefined()
- expect(result.data["description"]).toEqual("General coding and planning agent")
- expect(result.data["mode"]).toEqual("subagent")
- expect(result.data["model"]).toEqual("synthetic/hf:zai-org/GLM-4.7")
- expect(result.data["tools"]["write"]).toBeTrue()
- expect(result.data["tools"]["read"]).toBeTrue()
- expect(result.data["stuff"]).toBe("This is some stuff\n")
- expect(result.content.trim()).toBe("Strictly follow da rules")
- })
- })
|