Skip to content

Commit

Permalink
fmt: respect prettierrc and prettierignore (denoland/deno#3346)
Browse files Browse the repository at this point in the history
  • Loading branch information
axetroy authored and ry committed Nov 16, 2019
1 parent b383d25 commit e93216d
Show file tree
Hide file tree
Showing 14 changed files with 360 additions and 7 deletions.
179 changes: 172 additions & 7 deletions prettier/main.ts
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,8 @@
// This script formats the given source files. If the files are omitted, it
// formats the all files in the repository.
import { parse } from "../flags/mod.ts";
import * as path from "../path/mod.ts";
import * as toml from "../encoding/toml.ts";
import { ExpandGlobOptions, WalkInfo, expandGlob } from "../fs/mod.ts";
import { prettier, prettierPlugins } from "./prettier.ts";
const { args, cwd, exit, readAll, readFile, stdin, stdout, writeFile } = Deno;
Expand All @@ -40,6 +42,10 @@ Options:
it will output to stdout, Defaults to
false.
--ignore <path> Ignore the given path(s).
--ignore-path <auto|disable|path> Path to a file containing patterns that
describe files to ignore. Optional
value: auto/disable/filepath. Defaults
to null.
--stdin Specifies to read the code from stdin.
If run the command in a pipe, you do not
need to specify this flag.
Expand All @@ -49,6 +55,10 @@ Options:
parser for stdin. available parser:
typescript/babel/markdown/json. Defaults
to typescript.
--config <auto|disable|path> Specify the configuration file of the
prettier.
Optional value: auto/disable/filepath.
Defaults to null.
JS/TS Styling Options:
--print-width <int> The line length where Prettier will try
Expand Down Expand Up @@ -106,7 +116,7 @@ Example:
// Available parsers
type ParserLabel = "typescript" | "babel" | "markdown" | "json";

interface PrettierOptions {
interface PrettierBuildInOptions {
printWidth: number;
tabWidth: number;
useTabs: boolean;
Expand All @@ -120,6 +130,9 @@ interface PrettierOptions {
arrowParens: string;
proseWrap: string;
endOfLine: string;
}

interface PrettierOptions extends PrettierBuildInOptions {
write: boolean;
}

Expand Down Expand Up @@ -343,10 +356,134 @@ async function* getTargetFiles(
}
}

/**
* auto detect prettier configuration file and return config if file exist.
*/
async function autoResolveConfig(): Promise<PrettierBuildInOptions> {
const configFileNamesMap = {
".prettierrc.json": 1,
".prettierrc.yaml": 1,
".prettierrc.yml": 1,
".prettierrc.js": 1,
".prettierrc.ts": 1,
"prettier.config.js": 1,
"prettier.config.ts": 1,
".prettierrc.toml": 1
};

const files = await Deno.readDir(".");

for (const f of files) {
if (f.isFile() && configFileNamesMap[f.name]) {
const c = await resolveConfig(f.name);
if (c) {
return c;
}
}
}

return;
}

/**
* parse prettier configuration file.
* @param filepath the configuration file path.
* support extension name with .json/.toml/.js
*/
async function resolveConfig(
filepath: string
): Promise<PrettierBuildInOptions> {
let config: PrettierOptions = undefined;

function generateError(msg: string): Error {
return new Error(`Invalid prettier configuration file: ${msg}.`);
}

const raw = new TextDecoder().decode(await Deno.readFile(filepath));

switch (path.extname(filepath)) {
case ".json":
try {
config = JSON.parse(raw) as PrettierOptions;
} catch (err) {
throw generateError(err.message);
}
break;
case ".yml":
case ".yaml":
// TODO: Unimplemented loading yaml / yml configuration file yet.
break;
case ".toml":
try {
config = toml.parse(raw) as PrettierOptions;
} catch (err) {
throw generateError(err.message);
}
break;
case ".js":
case ".ts":
const absPath = path.isAbsolute(filepath)
? filepath
: path.join(cwd(), filepath);

try {
const output = await import(
// TODO: Remove platform condition
// after https://github.com/denoland/deno/issues/3355 fixed
Deno.build.os === "win" ? "file://" + absPath : absPath
);

if (output && output.default) {
config = output.default;
} else {
throw new Error(
"Prettier of JS version should have default exports."
);
}
} catch (err) {
throw generateError(err.message);
}

break;
default:
break;
}

return config;
}

/**
* auto detect .prettierignore and return pattern if file exist.
*/
async function autoResolveIgnoreFile(): Promise<string[]> {
const files = await Deno.readDir(".");

for (const f of files) {
if (f.isFile() && f.name === ".prettierignore") {
return await resolveIgnoreFile(f.name);
}
}

return [];
}

/**
* parse prettier ignore file.
* @param filepath the ignore file path.
*/
async function resolveIgnoreFile(filepath: string): Promise<string[]> {
const raw = new TextDecoder().decode(await Deno.readFile(filepath));

return raw
.split("\n")
.filter((v: string) => !!v.trim() && v.trim().indexOf("#") !== 0)
.map(v => v);
}

async function main(opts): Promise<void> {
const { help, ignore, check, _: args } = opts;
const { help, check, _: args } = opts;

const prettierOpts: PrettierOptions = {
let prettierOpts: PrettierOptions = {
printWidth: Number(opts["print-width"]),
tabWidth: Number(opts["tab-width"]),
useTabs: Boolean(opts["use-tabs"]),
Expand All @@ -368,10 +505,37 @@ async function main(opts): Promise<void> {
exit(0);
}

const files = getTargetFiles(
args.length ? args : ["."],
Array.isArray(ignore) ? ignore : [ignore]
);
const configFilepath = opts["config"];

if (configFilepath && configFilepath !== "disable") {
const config =
configFilepath === "auto"
? await autoResolveConfig()
: await resolveConfig(configFilepath);

if (config) {
prettierOpts = { ...prettierOpts, ...config };
}
}

let ignore = opts.ignore as string[];

if (!Array.isArray(ignore)) {
ignore = [ignore];
}

const ignoreFilepath = opts["ignore-path"];

if (ignoreFilepath && ignoreFilepath !== "disable") {
const ignorePatterns: string[] =
ignoreFilepath === "auto"
? await autoResolveIgnoreFile()
: await resolveIgnoreFile(ignoreFilepath);

ignore = ignore.concat(ignorePatterns);
}

const files = getTargetFiles(args.length ? args : ["."], ignore);

const tty = Deno.isTTY();

Expand All @@ -396,6 +560,7 @@ main(
parse(args.slice(1), {
string: [
"ignore",
"ignore-path",
"printWidth",
"tab-width",
"trailing-comma",
Expand Down
156 changes: 156 additions & 0 deletions prettier/main_test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -378,4 +378,160 @@ test(async function testPrettierReadFromStdin(): Promise<void> {
}
});

test(async function testPrettierWithAutoConfig(): Promise<void> {
const configs = [
"config_file_json",
"config_file_toml",
"config_file_js",
"config_file_ts"
];

for (const configName of configs) {
const cwd = join(testdata, configName);
const prettierFile = join(Deno.cwd(), "prettier", "main.ts");
const { stdout, stderr } = Deno.run({
args: [
execPath(),
"run",
"--allow-read",
"--allow-env",
prettierFile,
"../5.ts",
"--config",
"auto"
],
stdout: "piped",
stderr: "piped",
cwd
});

const output = decoder.decode(await Deno.readAll(stdout));
const errMsg = decoder.decode(await Deno.readAll(stderr));

assertEquals(
errMsg
.split(EOL)
.filter((line: string) => line.indexOf("Compile") !== 0)
.join(EOL),
""
);

assertEquals(output, `console.log('0');\n`);
}
});

test(async function testPrettierWithSpecifiedConfig(): Promise<void> {
interface Config {
dir: string;
name: string;
}
const configs: Config[] = [
{
dir: "config_file_json",
name: ".prettierrc.json"
},
{
dir: "config_file_toml",
name: ".prettierrc.toml"
},
{
dir: "config_file_js",
name: ".prettierrc.js"
},
{
dir: "config_file_ts",
name: ".prettierrc.ts"
}
];

for (const config of configs) {
const cwd = join(testdata, config.dir);
const prettierFile = join(Deno.cwd(), "prettier", "main.ts");
const { stdout, stderr } = Deno.run({
args: [
execPath(),
"run",
"--allow-read",
"--allow-env",
prettierFile,
"../5.ts",
"--config",
config.name
],
stdout: "piped",
stderr: "piped",
cwd
});

const output = decoder.decode(await Deno.readAll(stdout));
const errMsg = decoder.decode(await Deno.readAll(stderr));

assertEquals(
errMsg
.split(EOL)
.filter((line: string) => line.indexOf("Compile") !== 0)
.join(EOL),
""
);

assertEquals(output, `console.log('0');\n`);
}
});

test(async function testPrettierWithAutoIgnore(): Promise<void> {
// only format typescript file
const cwd = join(testdata, "ignore_file");
const prettierFile = join(Deno.cwd(), "prettier", "main.ts");
const { stdout, stderr } = Deno.run({
args: [
execPath(),
"run",
"--allow-read",
"--allow-env",
prettierFile,
"**/*",
"--ignore-path",
"auto"
],
stdout: "piped",
stderr: "piped",
cwd
});

assertEquals(decoder.decode(await Deno.readAll(stderr)), "");

assertEquals(
decoder.decode(await Deno.readAll(stdout)),
`console.log("typescript");\nconsole.log("typescript1");\n`
);
});

test(async function testPrettierWithSpecifiedIgnore(): Promise<void> {
// only format javascript file
const cwd = join(testdata, "ignore_file");
const prettierFile = join(Deno.cwd(), "prettier", "main.ts");
const { stdout, stderr } = Deno.run({
args: [
execPath(),
"run",
"--allow-read",
"--allow-env",
prettierFile,
"**/*",
"--ignore-path",
"typescript.prettierignore"
],
stdout: "piped",
stderr: "piped",
cwd
});

assertEquals(decoder.decode(await Deno.readAll(stderr)), "");

assertEquals(
decoder.decode(await Deno.readAll(stdout)),
`console.log("javascript");\nconsole.log("javascript1");\n`
);
});

runIfMain(import.meta);
1 change: 1 addition & 0 deletions prettier/testdata/5.ts
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
console.log("0" )
Loading

0 comments on commit e93216d

Please sign in to comment.