Skip to content

Commit c1aee77

Browse files
JooHyung Parkclaude
andcommitted
[ai-assisted] fix(mas): add app-sandbox to child entitlements
Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 240f36f commit c1aee77

2 files changed

Lines changed: 16 additions & 3 deletions

File tree

entitlements.child.plist

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,10 @@
22
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
33
<plist version="1.0">
44
<dict>
5-
<!-- Child helpers must inherit sandbox from parent.
6-
Do NOT declare app-sandbox again here, or helpers can crash with
7-
forbidden-sandbox-reinit in MAS/TestFlight builds. -->
5+
<!-- Mac App Store requires explicit app-sandbox declaration
6+
even for child helpers. We include both app-sandbox and inherit. -->
7+
<key>com.apple.security.app-sandbox</key>
8+
<true/>
89
<key>com.apple.security.inherit</key>
910
<true/>
1011
</dict>

forge.config.ts

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,18 @@ const config: ForgeConfig = {
123123
{ stdio: 'inherit' }
124124
);
125125
console.log(`[postPackage] ✅ Successfully re-signed: ${helperApp}`);
126+
127+
// Verify the entitlements were applied
128+
try {
129+
const verifyOutput = execSync(
130+
`codesign -d --entitlements - "${helperPath}" 2>&1`,
131+
{ encoding: 'utf8' }
132+
);
133+
console.log(`[postPackage] Verifying entitlements for ${helperApp}:`);
134+
console.log(verifyOutput);
135+
} catch (verifyError) {
136+
console.error(`[postPackage] ⚠️ Failed to verify entitlements for ${helperApp}:`, verifyError);
137+
}
126138
} catch (error) {
127139
console.error(`[postPackage] ❌ Failed to re-sign ${helperApp}:`, error);
128140
throw error;

0 commit comments

Comments
 (0)