Skip to content

run pear.main field (failing test)#1001

Open
davidmarkclements wants to merge 2 commits intomainfrom
run-pear-main-field
Open

run pear.main field (failing test)#1001
davidmarkclements wants to merge 2 commits intomainfrom
run-pear-main-field

Conversation

@davidmarkclements
Copy link
Contributor

No description provided.

@Didericis Didericis force-pushed the run-pear-main-field branch 2 times, most recently from 2f923a1 to c7a2b0b Compare February 3, 2026 04:36
@Didericis Didericis force-pushed the run-pear-main-field branch from c7a2b0b to 98b99c5 Compare February 3, 2026 05:17
}
await this.constructor.build(this, result.value)
// FIXME: modify pear-state instead
this.manifest.main = this.main
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Equivalent to this PR (can be removed once pear-state is updated) -> holepunchto/pear-state#6

const bundle = await app.pod.bundle(state.entrypoint)
const bundle = await app.pod.bundle(
// FIXME: temporary demonstration of fix, should be removed
flags.pre ? state.entrypoint : path.join(state.entrypoint, state.main)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

logic that sets state.entrypoint should be doing this join, I think

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants