Skip to content

Say which log type holds the pre-deploy output - #37

Merged
anurag merged 1 commit into
mainfrom
ag/pre-deploy-log-type
Sep 24, 2026
Merged

anurag merged 1 commit into
mainfrom
ag/pre-deploy-log-type

Conversation

@anurag

@anurag anurag commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

A follow-up to #34. AGENTS.md said that a type filter can lose the pre-deploy output, because the Render documentation does not give its type. A read of the logs API now gives the type.

What the logs API gives

The last deploy of the factory gateway has a build and a pre-deploy step. In the time range of that deploy:

Query Lines
no type filter, as fetchDeployLogs() reads 81
type=build 70
type=app 11, with the only 2 lines about the pre-deploy step

A static site gave its 5 build lines for type=build and with no filter.

So a build filter loses the pre-deploy output, and no filter gives both types. The read printed only counts, no log text.

Changes

  • AGENTS.md: do not add a type filter, and why.
  • app/render.ts: the comment of fetchDeployLogs() says the same. The code does not change.

Verification

  • npm run check passes: 376 tests.
  • The data is from a deploy that went live. No generated app with an API is in the workspace now, so no failed pre-deploy was read. A failed deploy should give the same types.

🤖 Generated with Claude Code

AGENTS.md said that a type filter can lose the pre-deploy output,
because the Render documentation does not give its type. A read of the
logs API on 2026-09-24 gives the type.

The last deploy of the factory gateway has a build and a pre-deploy
step. In its time range, the logs API gave 70 lines for type=build, 11
lines for type=app, and 81 lines with no type filter. The two lines
about the pre-deploy step came back only for type=app. A static site
gave its 5 build lines for type=build and with no filter.

So a build filter loses the pre-deploy output, and no filter gives both
types. AGENTS.md and the comment of fetchDeployLogs() now say so. The
code does not change.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@anurag
anurag merged commit 0ba39f4 into main Sep 24, 2026
3 checks passed
@anurag
anurag deleted the ag/pre-deploy-log-type branch September 24, 2026 14:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant