Skip to content

Agents guide.#1065

Merged
xeioex merged 2 commits into
nginx:masterfrom
xeioex:agents
May 21, 2026
Merged

Agents guide.#1065
xeioex merged 2 commits into
nginx:masterfrom
xeioex:agents

Conversation

@xeioex
Copy link
Copy Markdown
Contributor

@xeioex xeioex commented May 20, 2026

Prodded by #1064.

@VadimZhestikov
Copy link
Copy Markdown
Contributor

ES2020 exponentiation operator **

The exponentiation operator ** was introduced in ES2016 (ES7), not ES2020. This should read "ES2016 exponentiation
operator **".

"Function constructors (since 0.3.6)":

Minor. It would be nice clarify: "Dynamic function construction (new Function(...)) is supported since 0.3.6."

VadimZhestikov
VadimZhestikov previously approved these changes May 21, 2026
Copy link
Copy Markdown
Contributor

@VadimZhestikov VadimZhestikov left a comment

Choose a reason for hiding this comment

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

Couple issues, otherwise looks good to me

@github-project-automation github-project-automation Bot moved this from New to In Progress in NGINX OSS Unified Workspace May 21, 2026
The new top-level AGENTS.md (with CLAUDE.md symlinked to it) is the
canonical index for agent and contributor instructions. Detailed
guides live under docs/agent/:

  docs/agent/engine-dev.md   building, testing, debugging the engine
                             and the nginx modules.
  docs/agent/js-dev.md       writing JavaScript for either engine,
                             with the common nginx API surface and
                             engine differences.
  docs/agent/js-dev-njs.md   specifics of the deprecated njs engine
                             and migration to QuickJS.
@xeioex
Copy link
Copy Markdown
Contributor Author

xeioex commented May 21, 2026

@VadimZhestikov

The exponentiation operator ** was introduced in ES2016 (ES7), not ES2020. This should read "ES2016 exponentiation operator **".

fixed.

"Function constructors (since 0.3.6)":

removed as too old.

Also added

  1. nginx.conf + JS integration points, added a note about who drives the execution (nginx)
  2. nginx.conf dirrectives
  3. make compaction and removed redundant text

@xeioex xeioex requested a review from VadimZhestikov May 21, 2026 00:35
Copy link
Copy Markdown
Contributor

@VadimZhestikov VadimZhestikov left a comment

Choose a reason for hiding this comment

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

Looks good

@xeioex xeioex merged commit 621f7cf into nginx:master May 21, 2026
2 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in NGINX OSS Unified Workspace May 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants