fix(packages/cli): watch top-level root dirs in diff command#53
Merged
zrosenbauer merged 5 commits intomainfrom Mar 21, 2026
Merged
fix(packages/cli): watch top-level root dirs in diff command#53zrosenbauer merged 5 commits intomainfrom
zrosenbauer merged 5 commits intomainfrom
Conversation
The diff command only watched specific subdirectories extracted from section includes, missing sibling directories like asset folders. Now it also watches the top-level root of each include path so images and other assets are detected. Also updates vercel.json ignoreCommand to use `pnpm zpress diff` instead of a hardcoded file list. Co-Authored-By: Claude <noreply@anthropic.com>
🦋 Changeset detectedLatest commit: ddbaca7 The changes in this PR will be included in the next version bump. This PR includes changesets to release 2 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (19)
📒 Files selected for processing (2)
💤 Files with no reviewable changes (2)
📝 WalkthroughWalkthroughThis pull request enhances the Changes
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~25 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
- Add zpress-vscode to changeset ignore list (has its own vsce workflow) - Set privatePackages version/tag to false to stop tagging examples - Remove private flag from vscode extension (excluded via ignore instead) Co-Authored-By: Claude <noreply@anthropic.com>
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@vercel.json`:
- Line 6: The ignoreCommand currently discards pnpm zpress diff's exit status by
evaluating its output with $(...), so failures are treated as empty output;
update the ignoreCommand to run pnpm zpress diff and preserve its exit code by
assigning its output to a variable (e.g., output="$(pnpm zpress diff)") and
immediately fail if the command exits non‑zero, then test emptiness of that
variable to decide deployment skipping; reference ignoreCommand and the pnpm
zpress diff invocation when making this change.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 918c19ba-c3a0-4561-889c-556be6bcc4aa
📒 Files selected for processing (3)
.changeset/diff-watch-asset-roots.mdpackages/cli/src/commands/diff.tsvercel.json
There was a problem hiding this comment.
Actionable comments posted: 8
🧹 Nitpick comments (1)
examples/kitchen-sink/.zpress/content/apps/web/routing.md (1)
35-42: Consider completing the middleware example.The middleware snippet shows configuration but omits the actual middleware function that performs authentication checks. Readers might find it difficult to understand how
publicPathsis used to protect routes.💡 Example of a more complete snippet
// middleware.ts +import { NextResponse } from 'next/server' +import type { NextRequest } from 'next/server' + const publicPaths = ['/auth/login', '/auth/signup'] +export function middleware(request: NextRequest) { + const { pathname } = request.nextUrl + + if (publicPaths.includes(pathname)) { + return NextResponse.next() + } + + // Check authentication here + // ... + + return NextResponse.redirect(new URL('/auth/login', request.url)) +} + export const config = { matcher: ['/((?!_next|api|static).*)'], }🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@examples/kitchen-sink/.zpress/content/apps/web/routing.md` around lines 35 - 42, The snippet defines publicPaths and config but omits the actual middleware handler that uses them; add an exported middleware function (export default function middleware(req)) that reads the request path (req.nextUrl.pathname), skips paths in publicPaths, performs an auth check (e.g., verify session cookie or token), and when unauthenticated redirects to '/auth/login' while preserving the return URL; ensure the function uses the existing publicPaths and config.matcher's behavior and returns NextResponse.next() for allowed requests and NextResponse.redirect(...) for blocked ones.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@examples/kitchen-sink/.zpress/content/apps/api/reference/openapi.json`:
- Around line 857-861: The global "bearerAuth" security definition is applied to
all operations, but the public endpoints /auth/login and /auth/refresh need to
override it; edit the OpenAPI operations for the paths "/auth/login" and
"/auth/refresh" in openapi.json and add an explicit operation-level "security":
[] entry to each operation (ensuring it sits alongside the existing
responses/requestBody/etc.), which will override the global security requirement
for those endpoints.
- Around line 24-29: Add explicit bounds to the pagination parameters and
response array: update the query parameter schemas for "limit" and "offset" (the
parameter objects named limit and offset) to include "minimum" and "maximum"
constraints (e.g., minimum 0 for offset, minimum 1 for limit and a sensible
maximum like 100 or 1000), and update the response schema where the "data" array
is defined to include "maxItems" (and optionally "minItems") to prevent
unbounded responses; ensure the changes are applied in openapi.json to the
parameter objects named "limit" and "offset" and to the response schema that has
the "data" array.
In `@examples/kitchen-sink/.zpress/content/apps/web.md`:
- Around line 22-30: The markdown code fence in the README snippet (the
unlabeled triple-backtick block showing the apps/web/ tree) lacks a language
label and triggers MD040; update the fenced block around the directory tree (the
block that starts with "apps/web/") to include a language label such as "text"
(i.e., change ``` to ```text) so the linter recognizes it as a text/code block.
In `@examples/kitchen-sink/.zpress/content/apps/web/routing.md`:
- Around line 48-50: The fenced code block showing the path
"src/app/products/[id]/page.tsx" is missing a language identifier; update the
markdown snippet in routing.md to use a plain text language tag (e.g., change
the triple backticks to ```text or ```plaintext) so the file path renders
correctly, ensuring the block that contains src/app/products/[id]/page.tsx is
updated accordingly.
- Around line 12-29: The fenced code block containing the file tree (the
triple-backtick block in routing.md) lacks a language identifier; update the
opening fence to include a plaintext language hint (e.g., ```text or
```plaintext) so the directory tree renders/accessibility improves, by editing
the opening ``` to ```text (or ```plaintext) for the block that starts with
"src/app/" and ends with the closing ```; no other content changes required.
In `@examples/kitchen-sink/.zpress/content/architecture.md`:
- Around line 12-21: In architecture.md update the two unlabeled fenced code
blocks (the ASCII tree and the ASCII diagram) to include a language label (use
"text") on the opening ``` fence so markdownlint MD040 is satisfied; locate the
two fence blocks in the file and change ``` to ```text for the apps/packages
tree and the web→ui→config diagram respectively, leaving the ASCII content
unchanged.
In `@examples/kitchen-sink/.zpress/content/packages/db.md`:
- Around line 31-42: The markdown code fence for the directory tree in the
packages/db example is missing a language tag (MD040); update the opening
triple-backtick before the tree (the block showing "packages/db/ ...
drizzle.config.ts") to include a language such as "text" (e.g., ```text) so the
code fence specifies a language and satisfies markdownlint MD040.
In `@examples/kitchen-sink/.zpress/content/packages/db/migrations.md`:
- Around line 37-42: The fenced code block showing the migrations tree is
missing a language label (triggering markdownlint MD040); update the fenced
block in examples/kitchen-sink/.zpress/content/packages/db/migrations.md by
changing the opening fence from ``` to a labeled fence such as ```text so the
block becomes a labeled code fence (e.g., use "text") to satisfy the linter.
---
Nitpick comments:
In `@examples/kitchen-sink/.zpress/content/apps/web/routing.md`:
- Around line 35-42: The snippet defines publicPaths and config but omits the
actual middleware handler that uses them; add an exported middleware function
(export default function middleware(req)) that reads the request path
(req.nextUrl.pathname), skips paths in publicPaths, performs an auth check
(e.g., verify session cookie or token), and when unauthenticated redirects to
'/auth/login' while preserving the return URL; ensure the function uses the
existing publicPaths and config.matcher's behavior and returns
NextResponse.next() for allowed requests and NextResponse.redirect(...) for
blocked ones.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 91d28867-a4bd-45b7-a3a8-faf3df1ee2a9
⛔ Files ignored due to path filters (12)
examples/kitchen-sink/.zpress/content/public/banner.svgis excluded by!**/*.svgexamples/kitchen-sink/.zpress/content/public/icon.svgis excluded by!**/*.svgexamples/kitchen-sink/.zpress/content/public/icons/guides.svgis excluded by!**/*.svgexamples/kitchen-sink/.zpress/content/public/icons/reference.svgis excluded by!**/*.svgexamples/kitchen-sink/.zpress/content/public/icons/security.svgis excluded by!**/*.svgexamples/kitchen-sink/.zpress/content/public/icons/standards.svgis excluded by!**/*.svgexamples/kitchen-sink/.zpress/content/public/logo.svgis excluded by!**/*.svgexamples/kitchen-sink/.zpress/content/public/logos/blaxel.pngis excluded by!**/*.pngexamples/kitchen-sink/.zpress/content/public/logos/nango.svgis excluded by!**/*.svgexamples/kitchen-sink/.zpress/content/public/logos/trigger.svgis excluded by!**/*.svgexamples/kitchen-sink/.zpress/content/public/logos/vercel.svgis excluded by!**/*.svgexamples/kitchen-sink/.zpress/public/icon.svgis excluded by!**/*.svg
📒 Files selected for processing (47)
.changeset/config.jsonexamples/kitchen-sink/.gitignoreexamples/kitchen-sink/.zpress/content/.generated/manifest.jsonexamples/kitchen-sink/.zpress/content/.generated/nav.jsonexamples/kitchen-sink/.zpress/content/.generated/sidebar.jsonexamples/kitchen-sink/.zpress/content/.generated/workspaces.jsonexamples/kitchen-sink/.zpress/content/apps.mdxexamples/kitchen-sink/.zpress/content/apps/api.mdexamples/kitchen-sink/.zpress/content/apps/api/reference/createteam.mdxexamples/kitchen-sink/.zpress/content/apps/api/reference/createuser.mdxexamples/kitchen-sink/.zpress/content/apps/api/reference/deleteuser.mdxexamples/kitchen-sink/.zpress/content/apps/api/reference/getuser.mdxexamples/kitchen-sink/.zpress/content/apps/api/reference/index.mdxexamples/kitchen-sink/.zpress/content/apps/api/reference/listteammembers.mdxexamples/kitchen-sink/.zpress/content/apps/api/reference/listteams.mdxexamples/kitchen-sink/.zpress/content/apps/api/reference/listusers.mdxexamples/kitchen-sink/.zpress/content/apps/api/reference/login.mdxexamples/kitchen-sink/.zpress/content/apps/api/reference/openapi.jsonexamples/kitchen-sink/.zpress/content/apps/api/reference/refreshtoken.mdxexamples/kitchen-sink/.zpress/content/apps/api/reference/updateuser.mdxexamples/kitchen-sink/.zpress/content/apps/web.mdexamples/kitchen-sink/.zpress/content/apps/web/deployment.mdexamples/kitchen-sink/.zpress/content/apps/web/routing.mdexamples/kitchen-sink/.zpress/content/architecture.mdexamples/kitchen-sink/.zpress/content/contributing.mdexamples/kitchen-sink/.zpress/content/contributing/guides.mdxexamples/kitchen-sink/.zpress/content/contributing/guides/code-style.mdexamples/kitchen-sink/.zpress/content/contributing/guides/pull-requests.mdexamples/kitchen-sink/.zpress/content/getting-started.mdexamples/kitchen-sink/.zpress/content/guides.mdxexamples/kitchen-sink/.zpress/content/guides/adding-a-package.mdexamples/kitchen-sink/.zpress/content/guides/deployment.mdexamples/kitchen-sink/.zpress/content/guides/local-development.mdexamples/kitchen-sink/.zpress/content/index.mdexamples/kitchen-sink/.zpress/content/integrations.mdxexamples/kitchen-sink/.zpress/content/integrations/stripe.mdexamples/kitchen-sink/.zpress/content/packages.mdxexamples/kitchen-sink/.zpress/content/packages/config.mdexamples/kitchen-sink/.zpress/content/packages/db.mdexamples/kitchen-sink/.zpress/content/packages/db/migrations.mdexamples/kitchen-sink/.zpress/content/packages/db/schema.mdexamples/kitchen-sink/.zpress/content/packages/ui.mdexamples/kitchen-sink/.zpress/content/packages/ui/components.mdexamples/kitchen-sink/.zpress/content/packages/ui/theming.mdexamples/simple/.gitignoreextensions/vscode/package.jsonpackages/config/schemas/schema.json
💤 Files with no reviewable changes (1)
- extensions/vscode/package.json
✅ Files skipped from review due to trivial changes (36)
- examples/kitchen-sink/.zpress/content/integrations/stripe.md
- examples/kitchen-sink/.zpress/content/apps/api.md
- examples/kitchen-sink/.zpress/content/apps/api/reference/refreshtoken.mdx
- examples/kitchen-sink/.gitignore
- examples/simple/.gitignore
- examples/kitchen-sink/.zpress/content/apps/api/reference/login.mdx
- examples/kitchen-sink/.zpress/content/apps/api/reference/index.mdx
- examples/kitchen-sink/.zpress/content/contributing.md
- examples/kitchen-sink/.zpress/content/apps/api/reference/updateuser.mdx
- examples/kitchen-sink/.zpress/content/apps/api/reference/listteams.mdx
- examples/kitchen-sink/.zpress/content/apps/api/reference/deleteuser.mdx
- examples/kitchen-sink/.zpress/content/apps/api/reference/listusers.mdx
- examples/kitchen-sink/.zpress/content/apps/api/reference/getuser.mdx
- examples/kitchen-sink/.zpress/content/guides.mdx
- examples/kitchen-sink/.zpress/content/apps/api/reference/createuser.mdx
- examples/kitchen-sink/.zpress/content/apps/web/deployment.md
- examples/kitchen-sink/.zpress/content/index.md
- examples/kitchen-sink/.zpress/content/packages/ui.md
- examples/kitchen-sink/.zpress/content/contributing/guides.mdx
- examples/kitchen-sink/.zpress/content/contributing/guides/pull-requests.md
- examples/kitchen-sink/.zpress/content/apps.mdx
- examples/kitchen-sink/.zpress/content/.generated/nav.json
- examples/kitchen-sink/.zpress/content/packages/ui/theming.md
- examples/kitchen-sink/.zpress/content/contributing/guides/code-style.md
- examples/kitchen-sink/.zpress/content/packages.mdx
- examples/kitchen-sink/.zpress/content/guides/local-development.md
- examples/kitchen-sink/.zpress/content/packages/ui/components.md
- examples/kitchen-sink/.zpress/content/.generated/manifest.json
- examples/kitchen-sink/.zpress/content/packages/config.md
- packages/config/schemas/schema.json
- examples/kitchen-sink/.zpress/content/guides/deployment.md
- examples/kitchen-sink/.zpress/content/.generated/workspaces.json
- examples/kitchen-sink/.zpress/content/apps/api/reference/createteam.mdx
- examples/kitchen-sink/.zpress/content/getting-started.md
- examples/kitchen-sink/.zpress/content/guides/adding-a-package.md
- examples/kitchen-sink/.zpress/content/packages/db/schema.md
Comment on lines
+24
to
+29
| "name": "limit", | ||
| "in": "query", | ||
| "schema": { | ||
| "type": "integer", | ||
| "default": 20 | ||
| } |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
FILE="examples/kitchen-sink/.zpress/content/apps/api/reference/openapi.json"
echo "Current limit schema:"
jq '.paths["/users"].get.parameters[] | select(.name=="limit").schema' "$FILE"
echo "Current users.data schema:"
jq '.paths["/users"].get.responses["200"].content["application/json"].schema.properties.data' "$FILE"Repository: joggrdocs/zpress
Length of output: 774
🏁 Script executed:
#!/bin/bash
set -euo pipefail
FILE="examples/kitchen-sink/.zpress/content/apps/api/reference/openapi.json"
# Get the file with line numbers to verify the exact lines mentioned
echo "=== File content with line numbers around the limit parameter ==="
sed -n '15,35p' "$FILE" | cat -n
echo -e "\n=== File content with line numbers around the data array ==="
sed -n '40,60p' "$FILE" | cat -n
# Check if there are other array properties or pagination parameters
echo -e "\n=== All parameters in /users endpoint ==="
jq '.paths["/users"].get.parameters[] | {name: .name, in: .in, schema: .schema}' "$FILE"
# Check all array-type properties in /users response
echo -e "\n=== All array-type properties in /users 200 response ==="
jq '.paths["/users"].get.responses["200"].content["application/json"].schema.properties | to_entries[] | select(.value.type == "array") | {property: .key, value: .value}' "$FILE"Repository: joggrdocs/zpress
Length of output: 2714
Add bounds to pagination parameters and response array to prevent unbounded responses.
The limit and offset query parameters lack minimum and maximum constraints, and the data array response lacks maxItems. This allows unbounded requests that could cause performance degradation.
Suggested fixes
{
"name": "limit",
"in": "query",
"schema": {
"type": "integer",
+ "minimum": 1,
+ "maximum": 100,
"default": 20
}
}{
"name": "offset",
"in": "query",
"schema": {
"type": "integer",
+ "minimum": 0,
"default": 0
}
}"data": {
"type": "array",
+ "maxItems": 100,
"items": {📝 Committable suggestion
‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
Suggested change
| "name": "limit", | |
| "in": "query", | |
| "schema": { | |
| "type": "integer", | |
| "default": 20 | |
| } | |
| "name": "limit", | |
| "in": "query", | |
| "schema": { | |
| "type": "integer", | |
| "minimum": 1, | |
| "maximum": 100, | |
| "default": 20 | |
| } |
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@examples/kitchen-sink/.zpress/content/apps/api/reference/openapi.json` around
lines 24 - 29, Add explicit bounds to the pagination parameters and response
array: update the query parameter schemas for "limit" and "offset" (the
parameter objects named limit and offset) to include "minimum" and "maximum"
constraints (e.g., minimum 0 for offset, minimum 1 for limit and a sensible
maximum like 100 or 1000), and update the response schema where the "data" array
is defined to include "maxItems" (and optionally "minItems") to prevent
unbounded responses; ensure the changes are applied in openapi.json to the
parameter objects named "limit" and "offset" and to the response schema that has
the "data" array.
Comment on lines
+857
to
+861
| "security": [ | ||
| { | ||
| "bearerAuth": [] | ||
| } | ||
| ] |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
set -euo pipefail
FILE="examples/kitchen-sink/.zpress/content/apps/api/reference/openapi.json"
echo "Global security:"
jq '.security' "$FILE"
echo "Operation-level security on auth endpoints (null means inherited global security):"
jq '.paths["/auth/login"].post.security, .paths["/auth/refresh"].post.security' "$FILE"Repository: joggrdocs/zpress
Length of output: 202
Add explicit unauthenticated security to /auth/login and /auth/refresh operations.
Global bearer auth is defined at lines 857-861, and both /auth/login and /auth/refresh inherit this security requirement since they lack explicit operation-level security definitions. Public authentication endpoints must override this with "security": [].
Suggested fixes
"/auth/login": {
"post": {
+ "security": [],
"operationId": "login",
"summary": "Authenticate","/auth/refresh": {
"post": {
+ "security": [],
"operationId": "refreshToken",
"summary": "Refresh token",🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@examples/kitchen-sink/.zpress/content/apps/api/reference/openapi.json` around
lines 857 - 861, The global "bearerAuth" security definition is applied to all
operations, but the public endpoints /auth/login and /auth/refresh need to
override it; edit the OpenAPI operations for the paths "/auth/login" and
"/auth/refresh" in openapi.json and add an explicit operation-level "security":
[] entry to each operation (ensuring it sits alongside the existing
responses/requestBody/etc.), which will override the global security requirement
for those endpoints.
Comment on lines
+22
to
+30
| ``` | ||
| apps/web/ | ||
| ├── src/ | ||
| │ ├── app/ # App router pages | ||
| │ ├── components/ # App-specific components | ||
| │ └── lib/ # Utilities and helpers | ||
| ├── public/ # Static assets | ||
| └── next.config.ts | ||
| ``` |
There was a problem hiding this comment.
Add a language label to the fenced block.
Line 22 is an unlabeled fence and triggers markdownlint MD040.
Suggested fix
-```
+```text
apps/web/
├── src/
│ ├── app/ # App router pages
│ ├── components/ # App-specific components
│ └── lib/ # Utilities and helpers
├── public/ # Static assets
└── next.config.ts</details>
<!-- suggestion_start -->
<details>
<summary>📝 Committable suggestion</summary>
> ‼️ **IMPORTANT**
> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
```suggestion
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 22-22: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@examples/kitchen-sink/.zpress/content/apps/web.md` around lines 22 - 30, The
markdown code fence in the README snippet (the unlabeled triple-backtick block
showing the apps/web/ tree) lacks a language label and triggers MD040; update
the fenced block around the directory tree (the block that starts with
"apps/web/") to include a language label such as "text" (i.e., change ``` to
```text) so the linter recognizes it as a text/code block.
Comment on lines
+12
to
+29
| ``` | ||
| src/app/ | ||
| ├── layout.tsx # Root layout (nav, providers) | ||
| ├── page.tsx # Home page | ||
| ├── dashboard/ | ||
| │ ├── layout.tsx # Dashboard layout (sidebar) | ||
| │ ├── page.tsx # Dashboard overview | ||
| │ ├── settings/ | ||
| │ │ └── page.tsx # User settings | ||
| │ └── @modal/ | ||
| │ └── (.)invite/ | ||
| │ └── page.tsx # Intercepted modal route | ||
| └── auth/ | ||
| ├── login/ | ||
| │ └── page.tsx | ||
| └── signup/ | ||
| └── page.tsx | ||
| ``` |
There was a problem hiding this comment.
Specify language for the fenced code block.
The directory tree code block is missing a language identifier. For plain text content like file trees, use text or plaintext to improve rendering and accessibility.
📝 Proposed fix
-```
+```text
src/app/
├── layout.tsx # Root layout (nav, providers)🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 12-12: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@examples/kitchen-sink/.zpress/content/apps/web/routing.md` around lines 12 -
29, The fenced code block containing the file tree (the triple-backtick block in
routing.md) lacks a language identifier; update the opening fence to include a
plaintext language hint (e.g., ```text or ```plaintext) so the directory tree
renders/accessibility improves, by editing the opening ``` to ```text (or
```plaintext) for the block that starts with "src/app/" and ends with the
closing ```; no other content changes required.
Comment on lines
+48
to
+50
| ``` | ||
| src/app/products/[id]/page.tsx | ||
| ``` |
There was a problem hiding this comment.
Specify language for the fenced code block.
The file path code block is missing a language identifier. Use text or plaintext for better rendering.
📝 Proposed fix
-```
+```text
src/app/products/[id]/page.tsx</details>
<details>
<summary>🧰 Tools</summary>
<details>
<summary>🪛 markdownlint-cli2 (0.21.0)</summary>
[warning] 48-48: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
</details>
</details>
<details>
<summary>🤖 Prompt for AI Agents</summary>
Verify each finding against the current code and only fix it if needed.
In @examples/kitchen-sink/.zpress/content/apps/web/routing.md around lines 48 -
50, The fenced code block showing the path "src/app/products/[id]/page.tsx" is
missing a language identifier; update the markdown snippet in routing.md to use
a plain text language tag (e.g., change the triple backticks to ```text or
contains src/app/products/[id]/page.tsx is updated accordingly.
Comment on lines
+12
to
+21
| ``` | ||
| apps/ | ||
| ├── web/ # Next.js frontend | ||
| └── api/ # Hono REST API | ||
|
|
||
| packages/ | ||
| ├── ui/ # Shared React components | ||
| ├── db/ # Database client and schema | ||
| └── config/ # Shared configuration | ||
| ``` |
There was a problem hiding this comment.
Label both fenced code blocks with a language.
Line 12 and Line 25 use unlabeled fences and trigger markdownlint MD040.
Suggested fix
-```
+```text
apps/
├── web/ # Next.js frontend
└── api/ # Hono REST API
packages/
├── ui/ # Shared React components
├── db/ # Database client and schema
└── config/ # Shared configuration
```diff
-```
+```text
web ──→ ui ──→ config
│ ↑
└──→ db ───────┘
↑
api ───┘
</details>
Also applies to: 25-31
<details>
<summary>🧰 Tools</summary>
<details>
<summary>🪛 markdownlint-cli2 (0.21.0)</summary>
[warning] 12-12: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
</details>
</details>
<details>
<summary>🤖 Prompt for AI Agents</summary>
Verify each finding against the current code and only fix it if needed.
In @examples/kitchen-sink/.zpress/content/architecture.md around lines 12 - 21,
In architecture.md update the two unlabeled fenced code blocks (the ASCII tree
and the ASCII diagram) to include a language label (use "text") on the opening
file and change ``` to ```text for the apps/packages tree and the web→ui→config
diagram respectively, leaving the ASCII content unchanged.
Comment on lines
+31
to
+42
| ``` | ||
| packages/db/ | ||
| ├── src/ | ||
| │ ├── index.ts # Client export | ||
| │ ├── schema/ # Table definitions | ||
| │ │ ├── users.ts | ||
| │ │ ├── products.ts | ||
| │ │ └── orders.ts | ||
| │ └── seed.ts # Development seed data | ||
| ├── migrations/ # SQL migration files | ||
| └── drizzle.config.ts | ||
| ``` |
There was a problem hiding this comment.
Specify a language for the directory tree code fence.
Line 31 should include a fence language to satisfy markdownlint MD040.
Suggested fix
-```
+```text
packages/db/
├── src/
│ ├── index.ts # Client export
│ ├── schema/ # Table definitions
│ │ ├── users.ts
│ │ ├── products.ts
│ │ └── orders.ts
│ └── seed.ts # Development seed data
├── migrations/ # SQL migration files
└── drizzle.config.ts</details>
<!-- suggestion_start -->
<details>
<summary>📝 Committable suggestion</summary>
> ‼️ **IMPORTANT**
> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
```suggestion
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 31-31: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@examples/kitchen-sink/.zpress/content/packages/db.md` around lines 31 - 42,
The markdown code fence for the directory tree in the packages/db example is
missing a language tag (MD040); update the opening triple-backtick before the
tree (the block showing "packages/db/ ... drizzle.config.ts") to include a
language such as "text" (e.g., ```text) so the code fence specifies a language
and satisfies markdownlint MD040.
Comment on lines
+37
to
+42
| ``` | ||
| migrations/ | ||
| ├── 0000_initial.sql | ||
| ├── 0001_add_orders_table.sql | ||
| └── 0002_add_user_role.sql | ||
| ``` |
There was a problem hiding this comment.
Add a language label for the fenced block.
Line 37 uses an unlabeled fence, which triggers markdownlint MD040.
Suggested fix
-```
+```text
migrations/
├── 0000_initial.sql
├── 0001_add_orders_table.sql
└── 0002_add_user_role.sql</details>
<!-- suggestion_start -->
<details>
<summary>📝 Committable suggestion</summary>
> ‼️ **IMPORTANT**
> Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
```suggestion
🧰 Tools
🪛 markdownlint-cli2 (0.21.0)
[warning] 37-37: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.
In `@examples/kitchen-sink/.zpress/content/packages/db/migrations.md` around lines
37 - 42, The fenced code block showing the migrations tree is missing a language
label (triggering markdownlint MD040); update the fenced block in
examples/kitchen-sink/.zpress/content/packages/db/migrations.md by changing the
opening fence from ``` to a labeled fence such as ```text so the block becomes a
labeled code fence (e.g., use "text") to satisfy the linter.
These are generated outputs that should not be committed. Both example directories already have .zpress/ in their .gitignore. Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
zpress diffcommand now watches top-level root directories (e.g.docs/) in addition to specific subdirectories from section includes, so asset/image changes are detectedvercel.jsonignoreCommandto usepnpm zpress diffinstead of a hardcoded file listChanges
packages/cli/src/commands/diff.ts— AddedtoTopLevelRoothelper that extracts the first path segment from include directories.collectWatchPathsnow includes these roots so sibling dirs (likedocs/.assets/) are covered.vercel.json— Replaced manualgit diffpath list withpnpm zpress diffinignoreCommand.changeset/diff-watch-asset-roots.md— Patch changeset for@zpress/cliTesting
pnpm zpress diff --prettydetects changes indocs/.assets/(previously missed)Summary by CodeRabbit
New Features
diffcommand now monitors top-level root directories to detect asset and image changes more comprehensively.Chores