Skip to content

Commit e39b649

Browse files
mizchiclaude
andcommitted
fix: eliminate all project warnings from JS build
- Added "supported-targets": "js" to 6 packages that depend on JS-only libraries (luna, mars, node/*) - Prefixed 10 unused fields in actrun ActionManifestSpec with _ to suppress unused_field warnings Result: 0 warnings from project code. Only 1 external warning remains (yacc duplicate alias - not fixable). 85 tests pass. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent b5c06ee commit e39b649

7 files changed

Lines changed: 29 additions & 20 deletions

File tree

src/actrun/parser.mbt

Lines changed: 20 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1634,16 +1634,16 @@ priv struct ActionManifestSpec {
16341634
inputs : Map[String, String]
16351635
runtime : String
16361636
steps : Array[StepSpec]
1637-
main : String
1638-
pre : String?
1639-
post : String?
1640-
pre_if : String
1641-
post_if : String
1642-
image : String
1643-
args : Array[String]
1644-
entrypoint : String?
1645-
pre_entrypoint : String?
1646-
post_entrypoint : String?
1637+
_main : String
1638+
_pre : String?
1639+
_post : String?
1640+
_pre_if : String
1641+
_post_if : String
1642+
_image : String
1643+
_args : Array[String]
1644+
_entrypoint : String?
1645+
_pre_entrypoint : String?
1646+
_post_entrypoint : String?
16471647
}
16481648

16491649
///|
@@ -1768,16 +1768,16 @@ fn parse_action_manifest_yaml(text : String) -> ActionManifestParseResult {
17681768
inputs,
17691769
runtime: action_using,
17701770
steps,
1771-
main,
1772-
pre,
1773-
post,
1774-
pre_if,
1775-
post_if,
1776-
image,
1777-
args,
1778-
entrypoint,
1779-
pre_entrypoint,
1780-
post_entrypoint,
1771+
_main: main,
1772+
_pre: pre,
1773+
_post: post,
1774+
_pre_if: pre_if,
1775+
_post_if: post_if,
1776+
_image: image,
1777+
_args: args,
1778+
_entrypoint: entrypoint,
1779+
_pre_entrypoint: pre_entrypoint,
1780+
_post_entrypoint: post_entrypoint,
17811781
}),
17821782
errors,
17831783
}

src/adapters/mars_http/moon.pkg

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,3 +6,7 @@ import {
66
"mizchi/sol" @sol,
77
"bit-vcs/bithub/core" @core,
88
}
9+
10+
options(
11+
"supported-targets": "js",
12+
)

src/cmd/bithub/moon.pkg

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ import {
2323
} for "test"
2424

2525
options(
26+
"supported-targets": "js",
2627
is_main: true,
2728
link: { "js": { "format": "cjs" } },
2829
targets: {

src/cmd/bithub_bench/moon.pkg

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import {
66
}
77

88
options(
9+
"supported-targets": "js",
910
is_main: true,
1011
link: { "js": { "format": "cjs" } },
1112
targets: { "main.mbt": [ "js" ] },

src/cmd/main/moon.pkg

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import {
66
}
77

88
options(
9+
"supported-targets": "js",
910
is_main: true,
1011
link: { "js": { "format": "esm", "exports": ["fetch"] } },
1112
)

src/cmd/main_fetch_ssr/moon.pkg

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ import {
77
}
88

99
options(
10+
"supported-targets": "js",
1011
is_main: true,
1112
link: { "js": { "format": "cjs" } },
1213
targets: {

src/cmd/main_ssr/moon.pkg

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@ import {
66
}
77

88
options(
9+
"supported-targets": "js",
910
is_main: true,
1011
link: { "js": { "format": "cjs" } },
1112
targets: {

0 commit comments

Comments
 (0)