FilesExpand file tree

main

/

check-skill-import-boundaries.test.mjs

Copy path

Blame

More file actions

Blame

More file actions

Latest commit

 

History

History

History

59 lines (54 loc) · 2.07 KB

main

/

check-skill-import-boundaries.test.mjs

Copy path

Top

File metadata and controls

• Code
• Blame

59 lines (54 loc) · 2.07 KB

Raw

Copy raw file

Download raw file

Open symbols panel

Edit and raw actions

1

2

3

4

5

6

7

8

9

10

11

12

13

14

15

16

17

18

19

20

21

22

23

24

25

26

27

28

29

30

31

32

33

34

35

36

37

38

39

40

41

42

43

44

45

46

47

48

49

50

51

52

53

54

55

56

57

58

59

import assert from "node:assert/strict";

import { readFileSync, readdirSync } from "node:fs";

import { dirname, join, relative, resolve, sep } from "node:path";

import { it } from "node:test";

const parser = new Bun.Transpiler({ loader: "tsx" });

function moduleSpecifiers(source) {

return parser.scanImports(source.replace(/^#![^\n]*/, "")).map((entry) => entry.path);

}

function scriptFiles(dir) {

return readdirSync(dir, { withFileTypes: true }).flatMap((entry) => {

const path = join(dir, entry.name);

if (entry.isDirectory()) return scriptFiles(path);

return /\.(?:mjs|cjs|js|ts)$/.test(entry.name) && !/\.(?:test|spec)\./.test(entry.name)

? [path]

: [];

});

}

function escapingImports(root, file) {

return moduleSpecifiers(readFileSync(file, "utf8"))

.filter((specifier) => {

if (!specifier.startsWith(".")) return false;

const target = resolve(dirname(file), specifier);

const local = relative(root, target);

return local === ".." || local.startsWith(".." + sep);

})

.map((specifier) => relative(resolve("."), file) + ": " + specifier);

}

it("shipped scripts keep literal relative imports inside their own skill", () => {

const { skills } = JSON.parse(readFileSync("skills-manifest.json", "utf8"));

const issues = [];

let checked = 0;

for (const name of Object.keys(skills)) {

const root = resolve("skills", name);

const files = scriptFiles(root);

checked += files.length;

issues.push(...files.flatMap((file) => escapingImports(root, file)));

}

assert.ok(checked > 0, "must inspect shipped scripts");

assert.deepEqual(issues, []);

});

it("inspects static imports, re-exports, dynamic imports and require without reading comments", () => {

assert.deepEqual(

moduleSpecifiers(`

// import "ignored";

import "./side.mjs";

import { x } from "./named.mjs";

export { y } from "./export.mjs";

export * from "./star.mjs";

import("./dynamic.mjs");

require("./common.cjs");

`),

["./side.mjs", "./named.mjs", "./export.mjs", "./star.mjs", "./dynamic.mjs", "./common.cjs"],

);

});