do not include json reporter by default

Author: thenbe

README.md

@@ -47,6 +47,7 @@ use({
                -- end,
 
                -- env = {
+               --    PW_NVIM = "1",
                --    HELLO = "world",
                -- },
 
@@ -66,6 +67,36 @@ use({
 })
 ```
 
+## Configuration
+
+The only reporter required by `neotest-playwright` is the `json` reporter, which you need to set either in your `playwright.config.ts` or by using `extra_args`. Make sure _not_ to declare the json reporter's `outputFile` property in your config as this will be set by `neotest-playwright`.
+
+One way you can do this is by using environment variables.
+
+```lua
+-- init.lua
+
+require("neotest-playwright").adapter({
+   options = {
+      env = {
+         PW_NVIM = "1",
+      },
+   },
+})
+```
+
+```typescript
+// playwright.config.ts
+
+const config: PlaywrightTestConfig = {
+	reporter: process.env.PW_NVIM
+		? [['json'], ['list'], ['html', { open: 'never' }]] // only json is required. The rest are optional.
+		: [['list'], ['html', { open: 'never' }]], // Your default reporters.
+};
+```
+
+> Until `playwright` provides us a way to pass the `--reporters` flag without overwriting the `reporters` set in the user's config, we have to rely on the user handling this.
+
 ---
 
 ## Projects

src/build-command.ts

@@ -19,14 +19,13 @@ export type CommandOptionsPreset = Omit<CommandOptions, 'bin'>;
 /** A function that takes in CommandOptions and returns a string. */
 export const buildCommand = (options: CommandOptions, extraArgs: string[]) => {
 	const o = options;
-	const reporters = o.reporters ?? ['list', 'json'];
-	const reportersArg = buildReporters(reporters);
+	const reporters = o.reporters ? buildReporters(o.reporters) : null;
 
 	const command: string[] = [];
 
 	command.push(o.bin);
 	command.push('test');
-	if (reportersArg !== null) command.push(reportersArg);
+	if (reporters !== null) command.push(reporters);
 	if (o.debug === true) command.push('--debug');
 	if (o.headed === true) command.push('--headed');
 	if (o.retries !== undefined) command.push(`--retries=${o.retries}`);

src/playwright.ts

@@ -17,7 +17,6 @@ export const get_projects = () => {
 		{
 			bin: options.get_playwright_command(path),
 			config: options.get_playwright_config(path),
-			reporters: ['json'],
 			testFilter: './does-not-exist',
 		},
 		['--list'],