Skip to content

Conversation

@ainsleyclark
Copy link
Contributor

No description provided.

claude and others added 8 commits October 24, 2025 09:41
- Add lifecycle rules to B2 bucket module to keep only one file version
- Provision default B2 bucket automatically when webkit infra apply is called
- Set ACL to allPrivate for security
- Add comprehensive tests to verify bucket configuration
- Export bucket details via terraform outputs

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Document requirement to run go test ./... and pnpm check before committing
- Add step-by-step instructions for handling network issues during testing
- Explain GOTOOLCHAIN=local usage with temporary go.mod downgrade
- Emphasise never committing downgraded go.mod version

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Remove environment suffix from default bucket name (now just project name)
- Add variables for lifecycle rule configuration in b2 bucket module
- Use dynamic block for conditional lifecycle rule creation
- Pass lifecycle variables through resources module for user-defined buckets
- Update test to reflect simplified bucket naming

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Change lifecycle_rule to lifecycle_rules (plural) to match B2 provider schema
- Remove validation requiring apps or resources since default bucket is always provisioned
- Update test to use correct field name and initialise empty slices

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Add valid Project definition to tests (fixes empty bucket_name error)
- Change test expectations from error to success
- Verify default B2 bucket is created in plan
- Default bucket is NOT added to GitHub secrets (only user-defined resources are)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@codecov
Copy link

codecov bot commented Oct 24, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.53%. Comparing base (7f6b060) to head (7881923).
⚠️ Report is 32 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
- Coverage   64.59%   63.53%   -1.06%     
==========================================
  Files         154      154              
  Lines        6064     5071     -993     
==========================================
- Hits         3917     3222     -695     
+ Misses       2064     1761     -303     
- Partials       83       88       +5     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ainsleyclark ainsleyclark merged commit 6e66f86 into main Oct 24, 2025
4 of 5 checks passed
@ainsleyclark ainsleyclark deleted the claude/webkit-project-infra-011CURoCukEMTrSjppkDcAbp branch October 24, 2025 12:47
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.

3 participants