Skip to content
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
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions .env.example
Original file line number Diff line number Diff line change
Expand Up @@ -11,3 +11,9 @@ VITE_API_URL=
# Dev mode — shows the "Dev Login" button that bypasses Google OAuth.
# Set to "true" locally. NEVER set in Vercel env vars.
VITE_DEV_MODE=true

# ── PostHog Analytics ─────────────────────────────────────────────────────────
# Leave blank locally — PostHog is silently disabled when this key is missing.
# Set both vars in your Vercel / production environment dashboard.
VITE_POSTHOG_KEY=
VITE_POSTHOG_HOST=https://us.i.posthog.com
40 changes: 0 additions & 40 deletions .github/ISSUE_TEMPLATE/bug_report.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,50 +12,10 @@ body:
---

Before submitting, search open issues to avoid duplicates.
Vague reports without reproduction steps will be closed.

- type: textarea
id: description
attributes:
label: Description
description: A clear description of the bug.
validations:
required: true

- type: textarea
id: steps
attributes:
label: Steps to Reproduce
description: Exact steps to reproduce the behavior.
placeholder: |
1. Go to '...'
2. Click on '...'
3. See error
validations:
required: true

- type: textarea
id: expected
attributes:
label: Expected Behavior
description: What you expected to happen.
validations:
required: true

- type: textarea
id: environment
attributes:
label: Environment
placeholder: |
OS: macOS 14 / Windows 11 / Ubuntu 22.04
Browser: Chrome 124 / Firefox 125
Node.js: 20.x
Python: 3.12.x
validations:
required: true

- type: textarea
id: additional
attributes:
label: Additional Context
description: Screenshots, error logs, or any other relevant information.
26 changes: 3 additions & 23 deletions .github/ISSUE_TEMPLATE/feature_request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -12,29 +12,9 @@ body:
- A PR opened for a feature without a linked, approved issue will be closed.
- Check the [Roadmap in DOCUMENTATION.md](https://github.com/jpdevhub/FreshScanAi/blob/main/DOCUMENTATION.md#14-future-roadmap) first — if your idea is already planned, comment on the existing tracking issue instead.
- type: textarea
id: problem
id: description
attributes:
label: Problem
description: What problem does this feature solve?
label: Description
description: What problem does this feature solve and what is your proposed solution?
validations:
required: true

- type: textarea
id: solution
attributes:
label: Proposed Solution
description: Describe the feature you want to see implemented.
validations:
required: true

- type: textarea
id: alternatives
attributes:
label: Alternatives Considered
description: Any alternative approaches you have considered.

- type: textarea
id: additional
attributes:
label: Additional Context
description: Mockups, references, or any other relevant context.
26 changes: 0 additions & 26 deletions .github/ISSUE_TEMPLATE/good_first_issue.yml
Original file line number Diff line number Diff line change
Expand Up @@ -6,36 +6,10 @@ body:
attributes:
value: |
This issue is intended for contributors new to the project. It should be self-contained and completable without deep knowledge of the codebase.

- type: textarea
id: description
attributes:
label: Task Description
description: A clear description of what needs to be done.
validations:
required: true

- type: textarea
id: context
attributes:
label: Context and Background
description: Any background information or links to relevant code the contributor needs to get started.
validations:
required: true

- type: textarea
id: acceptance
attributes:
label: Acceptance Criteria
description: Define what done looks like for this task.
placeholder: |
- [ ] Criterion 1
- [ ] Criterion 2
validations:
required: true

- type: textarea
id: hints
attributes:
label: Hints
description: Relevant files, functions, or documentation to look at first.
23 changes: 23 additions & 0 deletions .github/workflows/pr-size-labeler.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
name: PR Size Labeler
on:
pull_request_target:
types: [opened, synchronize, reopened]

jobs:
size-label:
runs-on: ubuntu-latest
permissions:
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2: Missing contents: read permission required by the action. Add contents: read to the permissions block.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At .github/workflows/pr-size-labeler.yml, line 9:

<comment>Missing `contents: read` permission required by the action. Add `contents: read` to the permissions block.</comment>

<file context>
@@ -0,0 +1,23 @@
+jobs:
+  size-label:
+    runs-on: ubuntu-latest
+    permissions:
+      issues: write
+      pull-requests: write
</file context>

issues: write
pull-requests: write
steps:
- name: Add PR Size Label
uses: pascalgn/size-label-action@v0.5.5
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2: Pin to the latest release v0.5.7 instead of v0.5.5 to get the Node 24 runtime update, security fixes, and updated dependencies.

Prompt for AI agents
Check if this issue is valid — if so, understand the root cause and fix it. At .github/workflows/pr-size-labeler.yml, line 14:

<comment>Pin to the latest release v0.5.7 instead of v0.5.5 to get the Node 24 runtime update, security fixes, and updated dependencies.</comment>

<file context>
@@ -0,0 +1,23 @@
+      pull-requests: write
+    steps:
+      - name: Add PR Size Label
+        uses: pascalgn/size-label-action@v0.5.5
+        env:
+          GITHUB_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
</file context>

env:
GITHUB_TOKEN: "${{ secrets.GITHUB_TOKEN }}"
with:
sizes: >
{
"0": "level: easy",
"100": "level: medium",
"500": "level: hard"
}
24 changes: 0 additions & 24 deletions .github/workflows/stale.yml

This file was deleted.

4 changes: 2 additions & 2 deletions CONTRIBUTING.md
Original file line number Diff line number Diff line change
Expand Up @@ -126,7 +126,7 @@ Do not open a PR if any of these are failing locally. Fix them first.
### Manual requirements (checked during review)

- [ ] Branch is up to date with `main` — rebase, do not merge
- [ ] No `.env` files, credentials, secrets, or API keys are present anywhere in the diff
- [ ] No `.env` or `.env.local` files, credentials, secrets, or API keys are present anywhere in the diff
- [ ] No model weight files (`.pth`, `.pt`, `.onnx`, `.bin`) are committed
- [ ] No `__pycache__/` directories, `.pyc` files, or macOS `._*` metadata files are committed
- [ ] New endpoints include corresponding tests in `backend/tests/` or `backend/test_*.py`
Expand Down Expand Up @@ -192,7 +192,7 @@ PRs will be closed without review if they:

- Are submitted without a prior issue or maintainer approval for non-trivial changes
- Fail any automated CI gate
- Contain `.env` files, secrets, model weights, or binary artifacts
- Contain `.env` or `.env.local` files, secrets, model weights, or binary artifacts
- Touch the deployment configuration (`vercel.json`, `Dockerfile`, `startup.sh`) without approval
- Introduce a new dependency (npm or PyPI) without prior discussion
- Rewrite existing, working logic without a documented reason
Expand Down
6 changes: 3 additions & 3 deletions DOCUMENTATION.md
Original file line number Diff line number Diff line change
Expand Up @@ -862,9 +862,9 @@ npm run dev

| Mode | Endpoint | Models Required | Use Case |
|------|---------|----------------|----------|
| Demo | `/api/v1/scan-auto` | No | Demo presentations, CI |
| Auto (real) | `/api/v1/scan-auto` | Yes | Single-image mobile scan |
| Full scan | `/api/v1/scan` | Yes | Multi-region expert mode |
| Demo | `/api/v1/scan-auto` | No | Demo presentations, CI |
| Auto (real) | `/api/v1/scan-auto` | Yes | Single-image mobile scan |
| Full scan | `/api/v1/scan` | Yes | Multi-region expert mode |

### Production Deployment
- Frontend: Vercel / Netlify (static export)
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -116,7 +116,7 @@ DEV_BYPASS_AUTH=true # never set true in production

| Script | Description |
|---|---|
| `npm run setup` | Install dependencies, write `.env`, optionally start local Supabase |
| `npm run setup` | Install dependencies, write `.env.local` (frontend) & `.env` (backend), optionally start local Supabase |
| `npm run dev` | Start frontend and backend concurrently |
| `npm run build` | Production build (TypeScript + Vite) |
| `npm run lint` | Run ESLint across the codebase |
Expand Down
Loading