-
Notifications
You must be signed in to change notification settings - Fork 1
feat: add ESBuild-based edge bundler #94
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
trieloff
wants to merge
49
commits into
main
Choose a base branch
from
feat/esbuild-bundler
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
49 commits
Select commit
Hold shift + click to select a range
ca77df3
feat!: replace deprecated Dictionary with SecretStore and ConfigStore
claude a34e771
Merge remote-tracking branch 'origin/main' into replace-dictionary-wi…
trieloff eaa03fb
fix: resolve linting errors
claude abfb453
fix: use fetch() instead of non-existent _fastly.request() in linkRes…
claude d839332
feat: use separate SecretStores for action and package parameters
claude 9fc2f1a
feat!: remove gateway fallback mechanism
claude 1b26615
feat!: remove obsolete parameter management from FastlyGateway
claude 028e543
feat: update @adobe/fastly-native-promises to 3.1.0
cc27a8f
refactor: replace home-grown patches with new fastly-native-promises …
18193ac
refactor: remove unnecessary wrapper functions in ComputeAtEdgeDeployer
ebf2400
fix: add no-op updatePackage method to FastlyGateway
e3babbd
fix: handle duplicate resource link errors gracefully
ef62ee2
fix: regenerate package-lock.json for fastly-native-promises 3.1.0
e31ff95
Merge branch 'main' into replace-dictionary-with-secretstore-configstore
b8175a5
fix: resolve eslint warnings for console statements
f5644c0
fix: improve error handling in fastly-adapter env proxy
dd13d00
feat: consolidate integration tests and add logging functionality
e1c642e
fix: replace unreliable httpbin.org with reliable www.aem.live endpoint
c156823
feat: populate context.func.name in Cloudflare adapter
0a75ad3
fix: never accept 503 responses in integration tests
f591c6b
fix: improve coverage reporting in CI to include all tests
ec9ea44
feat: add explicit credential validation for integration tests
51b8b3b
fix: remove hardcoded credentials and fix CI environment variable han…
4d06bf1
refactor: remove redundant edge-integration script
fd8da3c
cleanup: remove unused logging-example fixture and references
bc777bc
refactor: simplify CI workflow and remove unnecessary documentation
9debee7
refactor: move platform detection to edge-index.js and simplify adapters
claude 5ba10fe
feat: extract Fastly runtime helpers into separate mockable module
claude 0e31f27
feat: add fastly:* wildcard to webpack externals
claude 7c338f1
test: add esmock for proper Fastly adapter unit tests
claude 3b99cbe
fix: use @adobe/fetch CacheOverride and reliable URLs in fixture
claude 2e83554
test: add Integration keyword to platform test names
claude d903e4d
refactor: combine Cloudflare and Fastly deployments into single CLI call
claude 90dfa42
fix: add publicPath to webpack config for Fastly WASM runtime
claude 488ef19
fix: handle secret store race conditions and normalize names
claude c7a769a
fix: eliminate code splitting to support Fastly runtime
claude 241d7a7
feat: add ESBuild-based edge bundler
claude f38f67d
feat!: remove webpack-based EdgeBundler in favor of ESBuild
claude cf966df
fix: add mocha exit flag to prevent hanging tests
claude 48fab18
fix: simplify fetch polyfill and fix cloudflare adapter
claude caa98d2
test(edge-action): add environment detection test route
claude e8f8adc
feat(fastly): add dynamic backend support
claude 3ce95b3
feat(fetch): add automatic backend resolution for Fastly
claude 373d9ad
chore(edge-index): add debug logging for platform detection
claude 557872e
fix(fetch): reuse fastly-runtime.js for platform detection
claude 51ba84d
fix: address PR #94 review comments
claude e337df5
chore: update package-lock.json
claude 9d274bd
fix(bundler): improve validateBundle to start server and make HTTP re…
claude 1d2a3f6
fix(fastly): simplify dynamic backend to minimal config
claude File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
| # Environment variables for local development | ||
| # Copy this file to .env and fill in your actual credentials | ||
|
|
||
| # Fastly API token for Compute@Edge deployments | ||
| # Get this from: https://manage.fastly.com/account/personal/tokens | ||
| HLX_FASTLY_AUTH=your_fastly_api_token_here | ||
|
|
||
| # Cloudflare API token for Workers deployments | ||
| # Get this from: https://dash.cloudflare.com/profile/api-tokens | ||
| CLOUDFLARE_AUTH=your_cloudflare_api_token_here | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's a trailing space after "deployments" on line 8.