Skip to content

Conversation

@ryoppippi
Copy link
Member

@ryoppippi ryoppippi commented Jan 16, 2026

Summary

  • Add .claude/rules/nix-workflow.md with guidance on Nix development environment and CI configuration
  • Update CLAUDE.md to include the new rule in the available rules table

Details

Based on stackone-ai-node's nix-workflow.md, adapted for the Python SDK with:

  • flake-parts for modular flake configuration
  • treefmt-nix for unified formatting
  • git-hooks.nix for pre-commit hooks
  • Python-specific tooling (uv, ty, just)

Test plan

  • Verify the rule file is properly formatted
  • Confirm CLAUDE.md table is correctly updated

Summary by cubic

Add a nix-workflow rule detailing Nix dev environment and CI setup for the Python SDK (flake-parts, treefmt-nix, git-hooks, uv/ty/just, and setup-nix usage).
Update CLAUDE.md to include the new rule.

Written for commit a7762af. Summary will update on new commits.

- Add .claude/rules/nix-workflow.md with guidance on:
  - Using flake.nix with flake-parts for dev environment
  - Adding tools to buildInputs
  - Treefmt and git-hooks configuration
  - CI workflow with setup-nix action
  - Recommended build flags (--print-build-logs --show-trace)
- Update CLAUDE.md to include nix-workflow in available rules table

Based on stackone-ai-node's nix-workflow.md, adapted for Python SDK
with flake-parts, treefmt-nix, and git-hooks.nix specifics.
Copilot AI review requested due to automatic review settings January 16, 2026 21:03
Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 2 files

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds documentation for Nix workflow conventions to the Claude rules system. The new rule provides comprehensive guidance on using Nix for development environment setup and CI configuration.

Changes:

  • Added new .claude/rules/nix-workflow.md documenting Nix development environment setup, treefmt configuration, git hooks, and CI workflow
  • Updated CLAUDE.md to include nix-workflow in the available rules table

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.claude/rules/nix-workflow.md New rule documenting Nix flake usage, development tools, CI configuration, and build flags
CLAUDE.md Added nix-workflow entry to the Available Rules table

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@@ -0,0 +1,99 @@
# Nix Workflow
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Missing YAML frontmatter. All other rules in the .claude/rules/ directory have YAML frontmatter at the top of the file with description and alwaysApply fields. This frontmatter is used by Claude Code to understand when to apply the rule. Add frontmatter similar to other rules like:

---
description: Nix development environment and CI configuration. (project)
alwaysApply: true
---

Copilot uses AI. Check for mistakes.
## Notes

- The project uses flake-parts for modular flake configuration
- Git submodules are initialised automatically in dev shell and CI
Copy link

Copilot AI Jan 16, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Inconsistent spelling. The project uses American English spelling (e.g., "organization", "organize" in CLAUDE.md and other files). Change "initialised" to "initialized" to maintain consistency.

Suggested change
- Git submodules are initialised automatically in dev shell and CI
- Git submodules are initialized automatically in dev shell and CI

Copilot uses AI. Check for mistakes.
These flags are only needed for debugging and slow down normal builds.
@ryoppippi ryoppippi merged commit b10c164 into main Jan 16, 2026
10 checks passed
@ryoppippi ryoppippi deleted the docs/nix-workflow-rule branch January 16, 2026 21:08
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