Skip to content

fix: move checkout before github-app-auth in release workflows#19

Merged
nsheaps merged 1 commit intomainfrom
claude/fix-release-workflow-Me9aW
Mar 5, 2026
Merged

fix: move checkout before github-app-auth in release workflows#19
nsheaps merged 1 commit intomainfrom
claude/fix-release-workflow-Me9aW

Conversation

@nsheaps
Copy link
Copy Markdown
Owner

@nsheaps nsheaps commented Mar 5, 2026

Summary

  • Fixes the fatal: not in a git directory error causing all release workflows to fail
  • The github-app-auth action runs git config user.name (local config) which requires being inside a git repository, but it was running before actions/checkout
  • Moves actions/checkout before the auth step in all three affected workflows (auto-release.yml, release.yml, release-web.yml)
  • Uses persist-credentials: false on checkout so the auth action can set up its own credentials

Affected workflows

  • .github/workflows/auto-release.yml
  • .github/workflows/release.yml
  • .github/workflows/release-web.yml

Test plan

  • Auto Release workflow passes on push to main
  • Manual Release workflow runs successfully
  • Release Web workflow runs successfully

https://claude.ai/code/session_012KqhD52DDkXcJmK94VpSVz

…ory'

The github-app-auth action runs `git config user.name` (local, not
--global) which requires a .git directory. Moving checkout before auth
and using persist-credentials: false so the auth action can set up
credentials after the repo is cloned.

Fixes the "fatal: not in a git directory" error in all release workflows.

https://claude.ai/code/session_012KqhD52DDkXcJmK94VpSVz
@nsheaps nsheaps self-assigned this Mar 5, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 5, 2026

Release Version Check

🟡 MINOR version bump

Version
Current 0.0.1
Next 0.1.0

This version will be released automatically when this PR is merged to main.

Changelog preview

0.1.0 (2026-03-05)

Features

  • add Claude Code rules for UI evidence, task completion, GitHub access, and PR management (4520912)
  • add file browser, space switcher, disable imports, fix base path detection (4113a81)
  • add path-based SPA routing with 404.html redirect hack (c73c62e), closes #pageId
  • add release-it versioning with conventional changelog (2e59640)
  • bdd: create features/ directory structure with feature files [T0.9] (e4cd58d)
  • ci: add GitHub Pages preview deployment workflow [T0.11] (f2670d8)
  • core: add all abstraction interfaces — types only [T0.3] (3d4e5b2)
  • core: add relations and rollups engine for database properties (T4.10) (8590b86)
  • core: add WebFsBackend using File System Access API [P2.7] (6308066)
  • deploy to GitHub Pages on push to main and add live demo link (da797f0)
  • e2e: set up Playwright with smoke test [T0.6] (f46a8f6)
  • editor: add table rendering and slash command [P2.4d] (c24de18)
  • infra: initialize monorepo with Nx, bun workspaces, TypeScript [T0.1] (0439217)
  • infra: set up CI workflow, ESLint, Prettier, typecheck [T0.2] (93e2f23)
  • restructure sidebar, improve page header, add trash page view (32b2e2f)
  • show changelog preview in PR version check comment (d010406)
  • show dynamic version on about page with commit SHA for preview deploys (e3d3874)
  • T5.1 — GitHub OAuth App flow with device flow support (e8856c8)
  • T5.2 — Repo picker/creator UI component (c851397)
  • T5.3 — Auto-commit engine with debounce and batching (370e446)
  • T5.4 — Auto-branch strategy (single, per-device, per-session) (e911198)
  • T5.5 — Auto-merge + conflict resolution engine (0269e42)
  • T5.6 — Conflict notification + resolution UI (e639977)
  • T5.7 — Page history viewer with Git log + diff (b1386e5)
  • T5.8 — Sync engine with pull/push cycle (06c110a)
  • test: set up BDD tooling with @amiceli/vitest-cucumber [T0.8] (9f1c8f0)
  • test: set up Vitest with unit test [T0.7] (bb84641)
  • ui: add DatabaseBoardView with drag-and-drop Kanban columns (T4.3) (b1b9da9)
  • ui: add DatabaseCalendarView with month grid and event display (T4.4) (6c9a714)
  • ui: add DatabaseGalleryView with image cards and property previews (T4.6) (00ca342)
  • ui: add DatabaseListView with expandable row details (T4.7) (c4c05ad)
  • ui: add DatabaseMapView with location parsing and marker list (T4.5) (934d2c8)
  • ui: add DatabaseProvider and useDatabaseEngine hook [P2.5] (8634ed4)
  • ui: add DatabaseTableView with sorting, filtering, and cell formatting (T4.2) (87195d2)
  • ui: add folder view for pages with children [P2.4b] (84bf88e)
  • ui: add import/export dialogs for Notion, Obsidian, and page export [P2.10-P2.12] (d2062ed)
  • ui: add InlineDatabase TipTap extension for embedded database blocks (T4.8) (fe9e602)
  • ui: add landing page for GitHub Pages with guides and demo info [P2.4a] (d2b6267)
  • ui: add LinkedDatabaseView with view type tabs and name editing (T4.9) (6fb79b6)
  • ui: add page cover banner and icon support [P2.4c] (35572d2)
  • ui: add PropertyEditor for all database property types (T4.12) (44bff25)
  • ui: add SelectEditor and MultiSelectEditor with color tags (T4.11) (772dd52)
  • ui: add SpaceManager for multi-space support [P2.9] (e06266f)
  • ui: create UI package skeleton with React, TailwindCSS [T0.4] (79972bf)
  • ui: implement CEPT_DEMO_MODE with sample content [T0.12] (202795d)
  • ui: replace localStorage with BrowserFsBackend (keystone P2.3) (2d857df)
  • ui: wire page CRUD to storage backend — individual page files [P2.4] (876e7da)
  • ui: wire search index to CeptSearchIndex with TF-IDF scoring [P2.6] (1cf10bd)
  • use release-it tag flow instead of push-to-main for deploys (2915703)
  • web: create Vite entry point with dev server [T0.5] (d3d758f)
  • wire spaces/import/export into UI, fix toggle touch, add responsive E2E (e0d4fbf)

Bug Fixes

  • add d3 as proper dependency for KnowledgeGraph component (4e496e2)
  • auto-release on merge to main and add PR version comments (5669874)
  • checkout PR head ref to avoid detached HEAD in version check (44d4e03)
  • CI mise version specs and vitest project configuration (99c879e)
  • CI runs on all branches, add e2e to workspaces (1258c4a)
  • core,signaling: fix GitBackend node:fs hardcode, diff empty hunks, signaling entry point [P2.1] (32cf46b)
  • core: harden Markdown parser/serializer roundtrip, fix extractText [P2.2] (c7cffa2)
  • deploy to /cept/app/ with site-level 404 router for SPA deep links (be6fefe)
  • guard URL updates behind initialization, fix reload deep linking (9497a5c)
  • move checkout before github-app-auth to fix 'not in a git directory' (1cb94e8)
  • page title hover-only outline, no selected bg in recent, rename Move... (225b3ac)
  • release workflow failures and set initial version to 0.0.1 (68c27cc)
  • remove unused variable 'i' in graph-animation test to fix lint (d288275)
  • resolve CD pipeline failure and increase release verbosity (05515af)
  • restore @amiceli/vitest-cucumber dependency in package.json (5661f54)
  • restrict CI triggers to push-to-main and any PR (#18) (55ad125)
  • run vitest from root instead of per-package via NX (8835894)
  • update .claude/settings.json for unattended tool execution (c35b8b1)
  • web: stub node: imports in web build, update continue protocol (4d0799f)
  • wire up App integration layer, fix Tailwind build, fix layout (b7020f2)

Refactoring

  • DRY 404/index.html into shared .github/pages/ files (5c7461e)

Documentation

  • create feature spec stubs for Phase 1 features [T0.10] (4af7c9b)
  • mark P2.4f and P2.4g complete in TASKS.md [P2.4f, P2.4g] (313b9c4)
  • mark Phase 1 complete in TASKS.md (10b56d5)

Tests

  • editor: add table rendering tests and E2E coverage [P2.4e] (e29b842)

CI/CD

  • run CI on all pull requests regardless of base branch (e67a8fe)
  • run E2E tests on all branches and draft PRs (964da1b)

Chores

  • regenerate bun.lock after rebase (568ee3a)

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 5, 2026

Preview Deployment

The web app for this PR has been deployed:

Open Preview

Use this to verify the app works correctly, especially for dependency updates.

github-actions bot added a commit that referenced this pull request Mar 5, 2026
@nsheaps nsheaps marked this pull request as ready for review March 5, 2026 20:36
@nsheaps nsheaps merged commit fdf388f into main Mar 5, 2026
8 checks passed
@nsheaps nsheaps deleted the claude/fix-release-workflow-Me9aW branch March 5, 2026 20:36
github-actions bot added a commit that referenced this pull request Mar 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants