chore: Update eslint rule to require empty line after license notice. (#784)

We should make sure there is always an empty line after license notice,
otherwise it gets dropped during build if it happens to be before import
statement.

---------

Co-authored-by: Piotr Paulski <piotrpaulski@chromium.org>

Author: zyzyzyryxy

rollup.config.mjs

@@ -14,6 +14,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+
 /**
  * @fileoverview take from {@link https://github.com/GoogleChromeLabs/chromium-bidi/blob/main/rollup.config.mjs | chromium-bidi}
  * and modified to specific requirement.

scripts/eslint_rules/check-license-rule.js

@@ -24,6 +24,7 @@ export default {
     schema: [],
     messages: {
       licenseRule: 'Add license header.',
+      emptyLine: 'Add empty line after license header.',
     },
   },
   defaultOptions: [],
@@ -64,6 +65,22 @@ export default {
             header.value.includes('License') ||
             header.value.includes('Copyright'))
         ) {
+          const nextToken = sourceCode.getTokenAfter(header, {
+            includeComments: true,
+          });
+          if (
+            nextToken &&
+            nextToken.loc.start.line === header.loc.end.line + 1
+          ) {
+            context.report({
+              node: node,
+              loc: header.loc,
+              messageId: 'emptyLine',
+              fix(fixer) {
+                return fixer.insertTextAfter(header, '\n');
+              },
+            });
+          }
           return;
         }
 

scripts/eval_scenarios/console_test.ts

@@ -3,6 +3,7 @@
  * Copyright 2026 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
+
 import assert from 'node:assert';
 
 import type {TestScenario} from '../eval_gemini.ts';

scripts/eval_scenarios/emulation_test.ts

@@ -3,6 +3,7 @@
  * Copyright 2026 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
+
 import assert from 'node:assert';
 
 import type {TestScenario} from '../eval_gemini.ts';

scripts/eval_scenarios/input_test.ts

@@ -3,6 +3,7 @@
  * Copyright 2026 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
+
 import assert from 'node:assert';
 
 import type {TestScenario} from '../eval_gemini.ts';

scripts/eval_scenarios/navigation_test.ts

@@ -3,6 +3,7 @@
  * Copyright 2026 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
+
 import assert from 'node:assert';
 
 import type {TestScenario} from '../eval_gemini.ts';

scripts/eval_scenarios/network_test.ts

@@ -3,6 +3,7 @@
  * Copyright 2026 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
+
 import assert from 'node:assert';
 
 import type {TestScenario} from '../eval_gemini.ts';

scripts/eval_scenarios/performance_test.ts

@@ -3,6 +3,7 @@
  * Copyright 2026 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
+
 import assert from 'node:assert';
 
 import type {TestScenario} from '../eval_gemini.ts';

scripts/eval_scenarios/snapshot_test.ts

@@ -3,6 +3,7 @@
  * Copyright 2026 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
+
 import assert from 'node:assert';
 
 import type {TestScenario} from '../eval_gemini.ts';

src/formatters/SnapshotFormatter.ts

@@ -3,6 +3,7 @@
  * Copyright 2025 Google LLC
  * SPDX-License-Identifier: Apache-2.0
  */
+
 import type {TextSnapshot, TextSnapshotNode} from '../McpContext.js';
 
 export class SnapshotFormatter {