feat(build): add account id constant to plugins (#5811)

* feat(build): add account id constant to plugins

* chore: forgot the test 🤦

* chore: add missing snapshot

* chore: types

* chore: make tsc happy

Author: JGAntunes

Diff for: packages/build/src/core/constants.ts

@@ -63,6 +63,10 @@ export interface NetlifyPluginConstants {
    * the Netlify site ID
    */
   SITE_ID: string
+  /**
+   * the Netlify account ID
+   */
+  ACCOUNT_ID?: string
   /**
    * the Netlify API access token
    */
@@ -90,7 +94,7 @@ export const getConstants = async function ({
   edgeFunctionsDistDir,
   cacheDir,
   netlifyConfig,
-  siteInfo: { id: siteId },
+  siteInfo: { id: siteId, account_id: accountId },
   apiHost,
   token,
   mode,
@@ -118,6 +122,8 @@ export const getConstants = async function ({
     NETLIFY_BUILD_VERSION: ROOT_PACKAGE_JSON.version,
     // The Netlify Site ID
     SITE_ID: siteId,
+    // The Netlify Account ID
+    ACCOUNT_ID: accountId,
     // The Netlify API access token
     NETLIFY_API_TOKEN: token,
     // The Netlify API host

Diff for: packages/build/tests/constants/fixtures/account_id/manifest.yml

@@ -0,0 +1,2 @@
+name: test
+inputs: []

Diff for: packages/build/tests/constants/fixtures/account_id/netlify.toml

@@ -0,0 +1,2 @@
+[[plugins]]
+package = "./plugin"

Diff for: packages/build/tests/constants/fixtures/account_id/plugin.js

@@ -0,0 +1,3 @@
+export const onPreBuild = function ({ constants: { ACCOUNT_ID } }) {
+  console.log(ACCOUNT_ID)
+}

Diff for: packages/build/tests/constants/snapshots/tests.js.md

@@ -1,4 +1,4 @@
-# Snapshot report for `tests/constants/tests.js`
+# Snapshot report for `packages/build/tests/constants/tests.js`
 
 The actual snapshot is saved in `tests.js.snap`.
 
@@ -1197,6 +1197,60 @@ Generated by [AVA](https://avajs.dev).
     (Netlify Build completed in 1ms)␊
     Build step duration: Netlify Build completed in 1ms`
 
+## constants.ACCOUNT_ID
+
+> Snapshot 1
+
+    `␊
+    Netlify Build                                                 ␊
+    ────────────────────────────────────────────────────────────────␊
+    ␊
+    > Version␊
+      @netlify/build 1.0.0␊
+    ␊
+    > Flags␊
+      accountId: test-account␊
+      debug: true␊
+      repositoryRoot: packages/build/tests/constants/fixtures/account_id␊
+      testOpts:␊
+        pluginsListUrl: test␊
+        silentLingeringProcesses: true␊
+    ␊
+    > Current directory␊
+      packages/build/tests/constants/fixtures/account_id␊
+    ␊
+    > Config file␊
+      packages/build/tests/constants/fixtures/account_id/netlify.toml␊
+    ␊
+    > Resolved config␊
+      build:␊
+        publish: packages/build/tests/constants/fixtures/account_id␊
+        publishOrigin: default␊
+      plugins:␊
+        - inputs: {}␊
+          origin: config␊
+          package: ./plugin␊
+    ␊
+    > Context␊
+      production␊
+    ␊
+    > Loading plugins␊
+       - ./plugin@1.0.0 from netlify.toml␊
+    ␊
+    ./plugin (onPreBuild event)                                   ␊
+    ────────────────────────────────────────────────────────────────␊
+    ␊
+    test-account␊
+    ␊
+    (./plugin onPreBuild completed in 1ms)␊
+    Build step duration: ./plugin onPreBuild completed in 1ms␊
+    ␊
+    Netlify Build Complete                                        ␊
+    ────────────────────────────────────────────────────────────────␊
+    ␊
+    (Netlify Build completed in 1ms)␊
+    Build step duration: Netlify Build completed in 1ms`
+
 ## constants.IS_LOCAL local
 
 > Snapshot 1

Diff for: packages/build/tests/constants/snapshots/tests.js.snap

@@ -112,6 +112,11 @@ test('constants.SITE_ID', async (t) => {
   t.snapshot(normalizeOutput(output))
 })
 
+test('constants.ACCOUNT_ID', async (t) => {
+  const output = await new Fixture('./fixtures/account_id').withFlags({ accountId: 'test-account' }).runWithBuild()
+  t.snapshot(normalizeOutput(output))
+})
+
 test('constants.IS_LOCAL local', async (t) => {
   const output = await new Fixture('./fixtures/is_local').runWithBuild()
   t.snapshot(normalizeOutput(output))

Diff for: packages/build/tests/constants/tests.js

@@ -2,7 +2,15 @@ import { promises as fs } from 'fs'
 import { join, relative, resolve } from 'path'
 
 import { parse } from '@babel/parser'
-import type { BinaryExpression, CallExpression, Expression, PrivateName, TemplateLiteral, TSType } from '@babel/types'
+import type {
+  BinaryExpression,
+  CallExpression,
+  Expression,
+  PrivateName,
+  TemplateLiteral,
+  TSType,
+  Program,
+} from '@babel/types'
 
 import { nonNullable } from '../../../utils/non_nullable.js'
 
@@ -120,7 +128,7 @@ const parseSource = (source: string) => {
 
 // Parses a JS/TS source and returns the resulting AST. If there is a parsing
 // error, it will get swallowed and `null` will be returned.
-export const safelyParseSource = (source: string) => {
+export const safelyParseSource = (source: string): Program | null => {
   try {
     return parseSource(source)
   } catch {