Skip to content

Commit 4b16d8c

Browse files
committed
chore: resolve workspace cross-dependencies from source
Add resolve.conditions: ['source'] to vite and vitest configs so workspace deps are resolved from TypeScript source instead of dist. This allows build:modified and test:modified to run against changed packages without requiring their deps to be pre-built. - packages/javascript: - bundle @hawk.so/core instead of externalizing it - move @hawk.so/core to devDependencies - add AGPL-3.0-only to license allow list - add ssr.resolve.conditions for Vite 7 node/SSR environment - packages/sveltekit: - add resolve.conditions: ['source'] - packages/sveltekit/playground: - add resolve.conditions: ['source']
1 parent 4a3d4e2 commit 4b16d8c

File tree

5 files changed

+34
-4
lines changed

5 files changed

+34
-4
lines changed

packages/javascript/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -40,10 +40,10 @@
4040
},
4141
"homepage": "https://github.com/codex-team/hawk.javascript#readme",
4242
"dependencies": {
43-
"@hawk.so/core": "workspace:^",
4443
"error-stack-parser": "^2.1.4"
4544
},
4645
"devDependencies": {
46+
"@hawk.so/core": "workspace:^",
4747
"@hawk.so/types": "0.5.8",
4848
"@vitest/coverage-v8": "^4.0.18",
4949
"jsdom": "^28.0.0",

packages/javascript/vite.config.ts

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@ export default defineConfig(() => {
2626
fileName: 'hawk',
2727
},
2828
rollupOptions: {
29-
external: ['@hawk.so/core'],
3029
plugins: [
3130
license({
3231
thirdParty: {
@@ -37,8 +36,8 @@ export default defineConfig(() => {
3736
return false;
3837
}
3938

40-
// Allow MIT and Apache-2.0 licenses.
41-
return ['MIT', 'Apache-2.0'].includes(dependency.license);
39+
// Allow MIT, Apache-2.0, and AGPL-3.0-only (first-party @hawk.so packages).
40+
return ['MIT', 'Apache-2.0', 'AGPL-3.0-only'].includes(dependency.license);
4241
},
4342
failOnUnlicensed: true,
4443
failOnViolation: true,
@@ -56,11 +55,18 @@ export default defineConfig(() => {
5655
},
5756

5857
resolve: {
58+
conditions: ['source'],
5959
alias: {
6060
'@/types': path.resolve(__dirname, './src/types'),
6161
},
6262
},
6363

64+
ssr: {
65+
resolve: {
66+
conditions: ['source'],
67+
},
68+
},
69+
6470
server: {
6571
port: 3303,
6672
open: './example/index.html',

packages/javascript/vitest.config.ts

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,4 +16,10 @@ export default defineConfig({
1616
},
1717
conditions: ['source'],
1818
},
19+
20+
ssr: {
21+
resolve: {
22+
conditions: ['source'],
23+
},
24+
},
1925
});

packages/sveltekit/playground/vite.config.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,4 +6,12 @@ export default defineConfig({
66
server: {
77
host: true,
88
},
9+
resolve: {
10+
conditions: ['source'],
11+
},
12+
ssr: {
13+
resolve: {
14+
conditions: ['source'],
15+
},
16+
},
917
});

packages/sveltekit/vite.config.ts

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,16 @@ export default defineConfig(() => {
1414
external: ['sveltekit', '@hawk.so/javascript'],
1515
},
1616
},
17+
resolve: {
18+
conditions: ['source'],
19+
},
20+
21+
ssr: {
22+
resolve: {
23+
conditions: ['source'],
24+
},
25+
},
26+
1727
plugins: [
1828
dts({
1929
tsconfigPath: './tsconfig.json',

0 commit comments

Comments
 (0)