Seperate out the functional tests so we don't run them in PRs

[snus-kin]

No description provided.

[Copilot]

Pull Request Overview

This PR separates functional tests from unit tests to prevent functional tests from running in PR pipelines. Functional tests require external dependencies and environment variables, while unit tests are fast and can run in isolation.

• Adds separate configuration for functional vs unit test execution
• Updates CI workflow to only run unit tests for PR builds
• Creates dedicated workflow for functional tests with manual trigger

Reviewed Changes

Copilot reviewed 15 out of 17 changed files in this pull request and generated 3 comments.

Show a summary per file

File	Description
package.json	Adds separate test commands and improved npm scripts for different test types
__tests__/vitest.config.ts	Updates unit test configuration to exclude functional tests
__tests__/vitest.functional.config.ts	New configuration file dedicated to functional tests
__tests__/unit/main.test.ts	Converts integration tests to proper unit tests with mocking
__tests__/functional/main.functional.test.ts	Moves integration tests to functional test suite
.github/workflows/nodejs.yml	Updates main CI to only run unit tests
.github/workflows/functional-tests.yml	New workflow for functional tests with manual trigger
eslint.config.mjs	Improves ESLint configuration formatting and adds ignore patterns
.pre-commit-config.yaml	Adds pre-commit hooks configuration
.gitmodules	Configures protobuf submodule
README.md	Updates documentation for new test structure and development setup
src/athena/athena.ts	Adds new error code enums to protobuf-generated code

Comments suppressed due to low confidence (1)

package.json:1

• There's a spelling error in the PR title. It should be 'Separate' instead of 'Seperate'.

{

---

_{Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.}

The merge-base changed after approval.