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
3 changes: 2 additions & 1 deletion .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -68,4 +68,5 @@ count.txt
.secrets
.actrc
.secrets
generated/
generated/
.wrangler
Comment thread
kattsushi marked this conversation as resolved.
1 change: 1 addition & 0 deletions .npmrc
Original file line number Diff line number Diff line change
Expand Up @@ -5,3 +5,4 @@
# JSR Configuration (for Deno packages)
#@jsr:registry=https://npm.jsr.io/

hoist=false
2 changes: 1 addition & 1 deletion apps/cli/tsconfig.tsbuildinfo
Original file line number Diff line number Diff line change
@@ -1 +1 @@
{"root":["./src/main.ts","./src/commands/add.ts","./src/commands/create.ts","./src/commands/init.ts","./src/commands/registry.ts"],"version":"5.9.2"}
{"root":["./src/main.ts","./src/commands/add.ts","./src/commands/create.ts","./src/commands/init.ts","./src/commands/registry.ts"],"version":"5.9.3"}

@coderabbitai coderabbitai Bot Feb 8, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🧹 Nitpick | 🔵 Trivial

Confirm committing tsconfig.tsbuildinfo is intentional.
If this file isn’t meant to be versioned, consider ignoring it to avoid churn; if it is intentional for caching, document that decision.

🤖 Prompt for AI Agents
In `@apps/cli/tsconfig.tsbuildinfo` at line 1, The committed tsbuild info file
tsconfig.tsbuildinfo appears to be an auto-generated TypeScript incremental
build cache; decide whether it should be versioned: if not, remove
tsconfig.tsbuildinfo from the repo and add tsconfig.tsbuildinfo to .gitignore so
it no longer appears in commits (verify after removal that builds still work for
entries in "root" like src/main.ts and src/commands/*), otherwise add a short
note to the repository docs (e.g., CONTRIBUTING or README) explaining why
tsconfig.tsbuildinfo is intentionally versioned for caching and CI
reproducibility so reviewers understand the decision.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

it is

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@kattsushi, thanks for clarifying! I've noted that tsconfig.tsbuildinfo is intentionally versioned in this project.


✏️ Learnings added
Learnt from: kattsushi
Repo: devx-op/effectify PR: 37
File: apps/cli/tsconfig.tsbuildinfo:1-1
Timestamp: 2026-02-08T23:51:14.382Z
Learning: In the effectify repository, the file `apps/cli/tsconfig.tsbuildinfo` is intentionally committed and versioned (not ignored), as confirmed by the maintainer.

Note: Learnings are effective only in the context of similar code segments. To apply general coding guidelines, please configure review instructions. You can manage existing learnings in the app.

File renamed without changes.
File renamed without changes.
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
{
"name": "@effectify/node-auth-app",
"name": "@effectify/node-auth-example",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

⚠️ Potential issue | 🟠 Major

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Search for references to the old package name
rg -n "@effectify/node-auth-app" -S

Repository: devx-op/effectify

Length of output: 2584


Update all references to the old package name in documentation.
The package rename to @effectify/node-auth-example was not applied to the documentation. The old name @effectify/node-auth-app still appears in 26 references across:

  • apps/docs/src/content/docs/backend/index.mdx
  • apps/docs/src/content/docs/backend/installation.md
  • apps/docs/src/content/docs/backend/packages/node-auth-app.md
  • apps/docs/src/content/docs/backend/getting-started.md
  • Spanish equivalents in apps/docs/src/content/docs/es/backend/

Update package names, installation commands, import statements, and page titles in these files.

🤖 Prompt for AI Agents
In `@apps/node-auth-example/package.json` at line 2, Replace every occurrence of
the old package name '@effectify/node-auth-app' with the new
'@effectify/node-auth-example' across the docs (including titles, MDX
frontmatter, headings, and any filenames/links that reference the old name), and
update installation commands and import statements (npm/yarn/pnpm and ES/CJS
import examples) to use the new package name; also mirror these changes in the
Spanish translations and any package-specific docs/pages so links and filenames
(e.g., package docs page names) remain consistent with
'@effectify/node-auth-example'.

"version": "0.0.1",
"private": true,
"type": "module",
Expand Down
Original file line number Diff line number Diff line change
@@ -1,30 +1,30 @@
{
"name": "@effectify/node-auth-app",
"name": "@effectify/node-auth-example",
"$schema": "../../node_modules/nx/schemas/project-schema.json",
"sourceRoot": "apps/node-auth-app/src",
"sourceRoot": "apps/node-auth-example/src",
"projectType": "application",
"targets": {
"build": {
"executor": "@nx/js:tsc",
"outputs": ["{options.outputPath}"],
"defaultConfiguration": "production",
"options": {
"outputPath": "apps/node-auth-app/dist",
"main": "apps/node-auth-app/src/main.ts",
"tsConfig": "apps/node-auth-app/tsconfig.app.json",
"outputPath": "apps/node-auth-example/dist",
"main": "apps/node-auth-example/src/main.ts",
"tsConfig": "apps/node-auth-example/tsconfig.app.json",
"format": ["esm"],
"generatePackageJson": false,
"updateBuildableProjectDepsInPackageJson": false,
"additionalEntryPoints": [],
"assets": [
"apps/node-auth-app/src/assets",
"apps/node-auth-example/src/assets",
{
"input": "node_modules/better-sqlite3/build/Release",
"glob": "better_sqlite3.node",
"output": "./"
},
{
"input": "apps/node-auth-app",
"input": "apps/node-auth-example",
"glob": "sqlite.db",
"output": "./"
}
Expand All @@ -40,16 +40,16 @@
"defaultConfiguration": "development",
"dependsOn": ["build"],
"options": {
"buildTarget": "@effectify/node-auth-app:build",
"buildTarget": "@effectify/node-auth-example:build",
"runBuildTargetDependencies": true,
"runtimeArgs": ["--experimental-specifier-resolution=node"]
},
"configurations": {
"development": {
"buildTarget": "@effectify/node-auth-app:build:development"
"buildTarget": "@effectify/node-auth-example:build:development"
},
"production": {
"buildTarget": "@effectify/node-auth-app:build:production"
"buildTarget": "@effectify/node-auth-example:build:production"
}
}
}
Expand Down
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
13 changes: 0 additions & 13 deletions apps/react-app-router/index.html

This file was deleted.

33 changes: 0 additions & 33 deletions apps/react-app-router/package.json

This file was deleted.

8 changes: 0 additions & 8 deletions apps/react-app-router/project.json

This file was deleted.

6 changes: 0 additions & 6 deletions apps/react-app-router/src/lib/runtime.server.ts

This file was deleted.

16 changes: 0 additions & 16 deletions apps/react-app-router/src/main.tsx

This file was deleted.

27 changes: 0 additions & 27 deletions apps/react-app-router/src/routes.tsx

This file was deleted.

9 changes: 0 additions & 9 deletions apps/react-app-router/src/routes/_index.tsx

This file was deleted.

24 changes: 0 additions & 24 deletions apps/react-app-router/src/routes/_layout.tsx

This file was deleted.

8 changes: 0 additions & 8 deletions apps/react-app-router/src/routes/about.tsx

This file was deleted.

Loading