Skip to content
This repository was archived by the owner on Mar 2, 2026. It is now read-only.
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
19 changes: 9 additions & 10 deletions .github/workflows/claude-code-review.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,14 +17,14 @@ jobs:
# github.event.pull_request.user.login == 'external-contributor' ||
# github.event.pull_request.user.login == 'new-developer' ||
# github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR'

runs-on: ubuntu-latest
permissions:
contents: read
pull-requests: read
issues: read
id-token: write

steps:
- name: Checkout repository
uses: actions/checkout@v4
Expand All @@ -39,7 +39,7 @@ jobs:

# Optional: Specify model (defaults to Claude Sonnet 4, uncomment for Claude Opus 4)
# model: "claude-opus-4-20250514"

# Direct prompt for automated review (no @claude mention needed)
direct_prompt: |
Please review this pull request and provide feedback on:
Expand All @@ -48,31 +48,30 @@ jobs:
- Performance considerations
- Security concerns
- Test coverage

Be constructive and helpful in your feedback.

# Optional: Use sticky comments to make Claude reuse the same comment on subsequent pushes to the same PR
# use_sticky_comment: true

# Optional: Customize review based on file types
# direct_prompt: |
# Review this PR focusing on:
# - For TypeScript files: Type safety and proper interface usage
# - For API endpoints: Security, input validation, and error handling
# - For React components: Performance, accessibility, and best practices
# - For tests: Coverage, edge cases, and test quality

# Optional: Different prompts for different authors
# direct_prompt: |
# ${{ github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR' &&
# ${{ github.event.pull_request.author_association == 'FIRST_TIME_CONTRIBUTOR' &&
# 'Welcome! Please review this PR from a first-time contributor. Be encouraging and provide detailed explanations for any suggestions.' ||
# 'Please provide a thorough code review focusing on our coding standards and best practices.' }}

# Optional: Add specific tools for running tests or linting
# allowed_tools: "Bash(npm run test),Bash(npm run lint),Bash(npm run typecheck)"

# Optional: Skip review for certain conditions
# if: |
# !contains(github.event.pull_request.title, '[skip-review]') &&
# !contains(github.event.pull_request.title, '[WIP]')

13 changes: 6 additions & 7 deletions .github/workflows/claude.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,26 +39,25 @@ jobs:
# This is an optional setting that allows Claude to read CI results on PRs
additional_permissions: |
actions: read

# Optional: Specify model (defaults to Claude Sonnet 4, uncomment for Claude Opus 4)
# model: "claude-opus-4-20250514"

# Optional: Customize the trigger phrase (default: @claude)
# trigger_phrase: "/claude"

# Optional: Trigger when specific user is assigned to an issue
# assignee_trigger: "claude-bot"

# Optional: Allow Claude to run specific commands
# allowed_tools: "Bash(npm install),Bash(npm run build),Bash(npm run test:*),Bash(npm run lint:*)"

# Optional: Add custom instructions for Claude to customize its behavior for your project
# custom_instructions: |
# Follow our coding standards
# Ensure all new code has tests
# Use TypeScript for new files

# Optional: Custom environment variables for Claude
# claude_env: |
# NODE_ENV: test

4 changes: 2 additions & 2 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ yarn.lock
.env.*
!.env.example
dist.zip
_raw/manifest.json
_raw/react-devtools.js
**/_raw/manifest.json
**/_raw/react-devtools.js
yarn-error.log
dist2
dist.rar
Expand Down
10 changes: 0 additions & 10 deletions CLAUDE.md
Original file line number Diff line number Diff line change
Expand Up @@ -85,16 +85,6 @@ FRW-Extension/
- Isolated business logic
- Shared between UI and background where appropriate

### External Packages

Core functionality is provided by npm packages from the `frw-core` repository:

- **`@onflow/frw-shared`** - Types, constants, utilities
- **`@onflow/frw-core`** - Core wallet services and blockchain logic
- **`@onflow/frw-reducers`** - State management reducers
- **`@onflow/frw-data-model`** - Caching system
- **`@onflow/frw-extension-shared`** - Extension-specific utilities

## Environment Setup

1. **Prerequisites**:
Expand Down
96 changes: 0 additions & 96 deletions apps/extension/_raw/manifest.json

This file was deleted.

1 change: 0 additions & 1 deletion apps/extension/_raw/react-devtools.js

This file was deleted.

5 changes: 0 additions & 5 deletions apps/extension/eslint.config.base.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -70,11 +70,6 @@ export const baseConfig = {
{
groups: [['builtin', 'external'], 'internal', ['parent', 'sibling', 'index']],
pathGroups: [
{
pattern: '@onflow/frw-*/**',
group: 'external',
position: 'after',
},
{
pattern: '@/**',
group: 'internal',
Expand Down
74 changes: 37 additions & 37 deletions apps/extension/eslint.config.mjs
Original file line number Diff line number Diff line change
@@ -1,37 +1,37 @@
import { baseConfig, testConfig, buildConfig } from './eslint.config.base.mjs';
const config = [
// Base config for all files
{
ignores: [
'**/dist/**',
'**/node_modules/**',
'**/.git/**',
'**/coverage/**',
'**/playwright/**',
'**/playwright-report/**',
'**/test-results/**',
'**/_raw/**',
'**/storybook-static/**',
],
},
// JavaScript and TypeScript files
{
...baseConfig,
files: ['**/*.{js,jsx,ts,tsx}'],
},
// Test files specific config
{
...baseConfig,
...testConfig,
files: ['**/*.test.ts', '**/__tests__/**'],
},
// Build files specific config
{
...baseConfig,
...buildConfig,
files: ['build/**/*.{js,jsx,ts,tsx}', '.storybook/**/*.{js,jsx,ts,tsx}'],
},
];
export default config;
import { baseConfig, testConfig, buildConfig } from './eslint.config.base.mjs';

const config = [
// Base config for all files
{
ignores: [
'**/dist/**',
'**/node_modules/**',
'**/.git/**',
'**/coverage/**',
'**/playwright/**',
'**/playwright-report/**',
'**/test-results/**',
'**/_raw/**',
'**/storybook-static/**',
],
},
// JavaScript and TypeScript files
{
...baseConfig,
files: ['**/*.{js,jsx,ts,tsx}'],
},
// Test files specific config
{
...baseConfig,
...testConfig,
files: ['**/*.test.ts', '**/__tests__/**'],
},
// Build files specific config
{
...baseConfig,
...buildConfig,
files: ['build/**/*.{js,jsx,ts,tsx}', '.storybook/**/*.{js,jsx,ts,tsx}'],
},
];

export default config;
Loading
Loading