adobe / adobe/helix-universal

adapter: context.env should not contain process.env

Open
#17 5 comments 0 reactions 0 assignees View on GitHub
Dominant language
JavaScript
Stars
5
Forks
7
Avg merge
10h 16m
Merged PRs (30d)
4

Description

having the `process.env` variables mixed into the `context.env` object is kind of redundant and might be confusing.
`context.env` should only contain the information that is specific to the deployed environment.
an action can still use `process.env` explicitely.

Contributor guide

Open the contributing guide

Research direction

Start by locating the adapter code that assembles context.env and inspect how process.env is added. Confirm the intended separation between deployed-environment data and process.env, then verify that actions can still access process.env explicitly and update or add the relevant tests if they are present.

Written by the indexing model from the issue text.

Assessment

Tech stack
javascript
Domain
backend
Issue type
Refactor
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.