Skip to content

Improve AI coding config - #5

Merged
lars20070 merged 53 commits into
masterfrom
lars20070/morechanges
Mar 30, 2026
Merged

Improve AI coding config#5
lars20070 merged 53 commits into
masterfrom
lars20070/morechanges

Conversation

@lars20070

@lars20070 lars20070 commented Mar 4, 2026

Copy link
Copy Markdown
Owner

The pull request refines the AI coding config. It contains no functional changes of the pytest-assay package.

Added

  • Claude Code extension in VS Code
  • lat.md docs and Claude rules added to CodeRabbit code guidelines
  • allow lat.md MCP tools in Claude Code

Changed

  • bump version to 0.1.6
  • single .claude/CLAUDE.md file
  • use the public GitHub Action to run lat check

Fixed

  • use OLLAMA_HOST instead of hard coded URL in unit tests, tests now run correctly in dev container

Removed

  • deno from dev container

Summary by CodeRabbit

  • Chores

    • Project version bumped to 0.1.6; editor recommendations updated; dev container feature removed and EOF normalized; example runtime config now invokes native CLI.
  • Documentation

    • Added a new lat workflow/spec with usage/checklist and removed the older workflow guidance document.
  • Configuration

    • Permissions allowlist extended; knowledge-base file patterns expanded; CI lat-check step switched to a reusable action.
  • Tests

    • Test harness now reads Ollama host from an environment variable.

@coderabbitai

coderabbitai Bot commented Mar 4, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@lars20070 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 22 minutes and 34 seconds before requesting another review.

Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 22 minutes and 34 seconds.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 75785101-ef84-4eef-bd67-02b91fa6c6f0

📥 Commits

Reviewing files that changed from the base of the PR and between ce246ec and 53a6fa2.

📒 Files selected for processing (7)
  • .claude/CLAUDE.md
  • .claude/settings.json
  • .coderabbit.yaml
  • .github/workflows/lat-check.yaml
  • .mcp.json.example
  • CLAUDE.md
  • tests/_ollama.py
📝 Walkthrough

Walkthrough

Bumps package version, adds an editor recommendation, removes Deno from devcontainer, switches an example MCP server to call lat directly, adds lat tooling docs/settings, expands code-guideline patterns, updates CI to use a lat-check action, deletes a top-level CLAUDE doc, and makes a test env var configurable.

Changes

Cohort / File(s) Summary
Version
pyproject.toml
Project version updated from 0.1.50.1.6.
Editor config
.vscode/extensions.json
Added anthropic.claude-code to recommendations.
Devcontainer
.devcontainer/devcontainer.json
Removed ghcr.io/devcontainers-community/features/deno:1 (including pinned version); EOF newline added.
MCP example
.mcp.json.example
mcpServers.lat now uses "command": "lat" and removed the prior Node CLI path from args (keeps "mcp").
Claude docs & settings
.claude/CLAUDE.md, .claude/settings.json
Added extensive lat.md usage/checklist/docs in .claude/CLAUDE.md and added six mcp__lat__* permissions to .claude/settings.json.
Top-level doc removed
CLAUDE.md
Deleted entire file contents (removed prior lat.md guidance).
Coderabbit patterns
.coderabbit.yaml
Expanded filePatterns to include .claude/rules/*.md and lat.md/**/*.md.
CI workflow
.github/workflows/lat-check.yaml
Replaced manual Node setup and npm install -g lat.md with lars20070/lat-check-action@v1; updated checkout step reference.
Tests (env config)
tests/_ollama.py
Introduced import os and made OLLAMA_BASE_URL configurable via OLLAMA_HOST env var (defaults to previous value).

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

  • simplify logging #4: Also modifies the pyproject.toml project version field (package version bump).
  • Fix Context7 config #7: Previous sequential version bump in pyproject.toml (0.1.4→0.1.5), related to version history.
🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title "more changes" is vague and generic, providing no meaningful information about the specific modifications in the changeset. Use a descriptive title that captures the main changes, such as "Bump version to 0.1.6 and update dev environment" or similar.
✅ Passed checks (2 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch lars20070/morechanges

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@codecov

codecov Bot commented Mar 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.59%. Comparing base (cd6ea12) to head (53a6fa2).
⚠️ Report is 54 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master       #5   +/-   ##
=======================================
  Coverage   76.59%   76.59%           
=======================================
  Files           8        8           
  Lines         393      393           
  Branches       54       54           
=======================================
  Hits          301      301           
  Misses         86       86           
  Partials        6        6           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@lars20070

Copy link
Copy Markdown
Owner Author

@coderabbitai summary

@coderabbitai

coderabbitai Bot commented Mar 29, 2026

Copy link
Copy Markdown
✅ Actions performed

Summary regeneration triggered.

@lars20070

Copy link
Copy Markdown
Owner Author

@coderabbitai Why did you not update the version bump to 0.1.6 in the summary?

@coderabbitai

coderabbitai Bot commented Mar 29, 2026

Copy link
Copy Markdown
🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Check the current version in pyproject.toml
cat pyproject.toml | grep -i version

Length of output: 40


@lars20070 The summary was generated at an earlier point in the PR's lifecycle when the version was being bumped to 0.1.4. Subsequent commits pushed it further to 0.1.6, but the summary was not automatically refreshed after those changes. I can regenerate the summary now to reflect the current state of the PR.

✅ Actions performed

Summary regeneration triggered.

@lars20070

Copy link
Copy Markdown
Owner Author

@coderabbitai resume

@lars20070

Copy link
Copy Markdown
Owner Author

@coderabbitai full review

@lars20070 lars20070 changed the title more changes Improve AI coding config Mar 30, 2026
@coderabbitai

coderabbitai Bot commented Mar 30, 2026

Copy link
Copy Markdown
✅ Actions performed

Full review triggered.

@lars20070
lars20070 merged commit c16e996 into master Mar 30, 2026
9 checks passed
@coderabbitai

coderabbitai Bot commented Mar 30, 2026

Copy link
Copy Markdown
✅ Actions performed

Reviews resumed.

@lars20070
lars20070 deleted the lars20070/morechanges branch March 30, 2026 09:20
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.

1 participant