feat: add voight-kampff-test

[MatteoGabriele]

Summary by CodeRabbit

• Chores
  • Updated a dependency and adjusted internal import paths.
  • Removed several internal classification and scoring utilities.
  • No changes to public APIs or user-facing behavior.

[netlify]

✅ Deploy Preview for agentscan ready!

Name	Link
🔨 Latest commit	47bb64d
🔍 Latest deploy log	https://app.netlify.com/projects/agentscan/deploys/69b2ced7983b8700082e3452
😎 Deploy Preview	https://deploy-preview-23--agentscan.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c8fbaa94-1767-4d04-905f-e312d913b785

📥 Commits

Reviewing files that changed from the base of the PR and between f3071ee and 47bb64d.

📒 Files selected for processing (1)

• app/composables/useClassificationDetails.ts

---

📝 Walkthrough

Walkthrough

Replaced local Voight‑Kampff utilities with the external npm package voight-kampff-test: removed identifyReplicant, getClassificationDetails, and CONFIG local modules, added the package to dependencies, and updated imports in composables, server API, and tests to use the new package.

Changes

Cohort / File(s)	Summary
Imports Updated app/composables/useClassificationDetails.ts, app/composables/useVerifiedAutomations.ts, server/api/identify-replicant/[username].get.ts, test/verified-automations.test.ts	Switched import sources from local scoped paths (~~/shared/utils/voight-kampff-test/..., ~~/server/api/...) to the voight-kampff-test package; no runtime logic changes.
Removed Local Implementations shared/utils/voight-kampff-test/identify-replicant.ts, shared/utils/voight-kampff-test/classification-details.ts, shared/utils/voight-kampff-test/config.ts	Deleted implementations for identifyReplicant, getClassificationDetails, and CONFIG (large data/logic removed); functionality now expected from external package.
Dependency Added package.json	Added dependency voight-kampff-test@^2.0.0 to replace local utilities.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• refactor: split user page in components and composables #17: Replaces/removes local voight‑kampff utilities and updates imports to the external voight-kampff-test package.
• refactor: use classification #18: Modifies the Voight‑Kampff classification surface and aligns usages with the external package.
• feat: add custom agent action #20: Updates references to the voight‑kampff utilities to match the extracted package.

Suggested labels

agentscan:automation-signals

Poem

🐰 I hopped through code tonight,
Tossed old utils into flight,
Packaged neat, no more local mess,
New imports snug, I quietly bless.
🍃

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title 'feat: add voight-kampff-test' clearly reflects the main change: adding a new external package dependency and refactoring internal code to use it instead of local implementations.

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

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch refactor/add-voight-kampff-test

📝 Coding Plan for PR comments

• Generate coding plan

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[coderabbitai]

🧹 Nitpick comments (1)

app/composables/useClassificationDetails.ts (1)

1-13: Derive ClassificationDetails from the package API instead of mirroring it locally.

Now that the source of truth lives in voight-kampff-test, keeping a hand-written { label, description } shape here can drift on future package updates. Deriving the type from getClassificationDetails keeps this wrapper aligned.

Proposed refactor

 import { getClassificationDetails } from "voight-kampff-test";
 
-type ClassificationDetails = {
-  label: string;
-  description: string;
-};
+type ClassificationDetails = ReturnType<typeof getClassificationDetails>;

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@app/composables/useClassificationDetails.ts` around lines 1 - 13, Replace the
hand-written ClassificationDetails shape with a type derived from the package
API: use ReturnType<typeof getClassificationDetails> (or the appropriate utility
if getClassificationDetails is async) so the wrapper stays in sync; update the
import usage around getClassificationDetails and change the computed generic in
useClassificationDetails to use the derived type instead of the local { label,
description } declaration, keeping the function name useClassificationDetails
and references to getClassificationDetails unchanged.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@app/composables/useClassificationDetails.ts`:
- Around line 1-13: Replace the hand-written ClassificationDetails shape with a
type derived from the package API: use ReturnType<typeof
getClassificationDetails> (or the appropriate utility if
getClassificationDetails is async) so the wrapper stays in sync; update the
import usage around getClassificationDetails and change the computed generic in
useClassificationDetails to use the derived type instead of the local { label,
description } declaration, keeping the function name useClassificationDetails
and references to getClassificationDetails unchanged.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 4b099d6b-1a80-4860-b3a1-ed8ecfa095be

📥 Commits

Reviewing files that changed from the base of the PR and between af484d7 and f3071ee.

⛔ Files ignored due to path filters (1)

• package-lock.json is excluded by !**/package-lock.json

📒 Files selected for processing (8)

• app/composables/useClassificationDetails.ts
• app/composables/useVerifiedAutomations.ts
• package.json
• server/api/identify-replicant/[username].get.ts
• shared/utils/voight-kampff-test/classification-details.ts
• shared/utils/voight-kampff-test/config.ts
• shared/utils/voight-kampff-test/identify-replicant.ts
• test/verified-automations.test.ts

💤 Files with no reviewable changes (3)

• shared/utils/voight-kampff-test/classification-details.ts
• shared/utils/voight-kampff-test/config.ts
• shared/utils/voight-kampff-test/identify-replicant.ts