@ampproject/toolbox-linter

AMP Linter

Overview

A linter for AMP documents: reports errors and suspicious constructions such as images missing or incorrectly sized, missing CORS headers, or invalid metadata.

Usage

Command-line (local build):

# from the amp-toolbox root
$ npm install
$ npm run build # generates packages/linter/dist/cli.js
$ cd packages/linter
$ node dist/cli.js https://amp.dev/

Command-line (from npm):

$ npx @ampproject/toolbox-linter https://amp.dev/

Node:

const fs = require("fs");
const linter = require("@ampproject/toolbox-linter");
const cheerio = require("cheerio");

const body = fs.readFileSync("amp-dev.html");
const context = {
  $: cheerio.load(body),
  headers: {},
  url: "https://amp.dev/"
};

linter.MetaCharsetIsFirst(context).then(console.log);

dump-signedexchange

One test has a dependency on the dump-signedexchange go binary. If this is available (installation instructions) at additional check of the application/signed-exchange response will be performed.

Development

Commands/Scripts

These scripts can be invoked in the usual way by npm run XXX if npm install is run in this directory. They can also be invoked from the amp-toolbox root directory without installing locally by lerna run --scope '*/toolbox-linter' XXX. (lerna sets the PATH so that the required binaries are available.)

build

Populates the dist directory with the appropriate *.js and *.d.ts files. Note that tests are not included. This script is intended to be used when building the npm package.

transpile

Transpiles *.ts into *.js. Unlike build, tests are included, and the *.js files are output into the same directory as the corresponding *.ts. This script is intended to be used during development.

test

Runs the tests. Automatically runs transpile first.

lint

Checks the code for lint errors using prettier.

Suggested Development Workflow

1. Create stub rule in rules/, that always "fails". e.g. it always returns qqqqqq. It should extend the Rule class.
2. Write tests in tests/network.ts. (If HTTP requests are required; if not then create a directory in tests/local/MyNewTest-1 that contains a source.html (AMP HTML source) and expected.json (expected JSON output), and tests/local.js will automatically execute your "test".)
3. Run the test using npm test. If the fixtures can't be found, they will be generated automatically (via real network requests). Hopefully your test will fail.
4. Fix the implementation, and re-run the test.

Changelog

v2.5.1 (2020-06-09)

:rocket: Enhancement

• linter
  • #826 Updated og image check to include name attr (@Dbrown910)
• cache-list, cli, core, cors, lighthouse-plugin-amp, linter, optimizer-express, optimizer, runtime-fetch, runtime-version, update-cache
  • #814 Move hero image preload out of experimental (@sebastianbenz)
• cli, linter
  • #806 Sort linter output (@Dbrown910)

:bug: Bug Fix

• optimizer
  • #828 Do not preload hero images using srcset (@sebastianbenz)

:house: Internal

• Other
  • #829 Update dependency lerna to v3.22.1 (@renovate-bot)
• optimizer
  • #812 Update dependency @ampproject/toolbox-optimizer to v2.5.0 (@renovate-bot)

Committers: 2

• Damani (@Dbrown910)
• Sebastian Benz (@sebastianbenz)