Skip to content

Conversation

Copy link

Copilot AI commented Dec 9, 2025

Summary

Adds comprehensive build/test documentation in .github/copilot-instructions.md to reduce agent exploration time and prevent common CI failures. Covers critical setup requirements, validated build commands, project structure, and CI/CD workflows for both Go backend and TypeScript/React UI.

UI Changes

N/A - Documentation only

Checklist

  • Linked to issue(s) above by issue number (e.g. Closes #<issue-number>)
  • One problem per PR (no unrelated changes)
  • Lints pass; CI green
  • Tricky parts are commented in code
  • Backward compatible with existing device firmware (See DEVELOPMENT.md for details)

Key Content

Critical Requirements Documented:

  • static directory must exist before Go tests (primary CI failure cause)
  • Node.js 22.20.0 required (works with 20.x but shows warnings)
  • Docker needed for ARM cross-compilation unless buildkit installed locally

Validated Commands:

# Go backend
mkdir -p static && touch static/.gitkeep  # Required first
go vet ./...
go test ./...
make build_dev SKIP_UI_BUILD=1 SKIP_NATIVE_IF_EXISTS=1

# TypeScript/React UI  
cd ui
npm ci
npm run i18n:compile  # Required before build if translations changed
npm run lint
npm run build:device

CI/CD Workflows:

  • build.yml: Frontend → backend → tests (requires static directory)
  • golangci-lint.yml: Lints Go code (requires static directory created first)
  • ui-lint.yml: Lints TypeScript/React

Project Conventions:

  • Use logger package, not fmt.Print* or log.* (golangci-lint enforced)
  • All UI strings must be localized via m.* pattern (paraglide-js)
  • Import order: builtin → external → internal → parent → sibling (ESLint enforced)

Common Issues Addressed:

  • Static directory missing → "pattern all:static: no matching files found"
  • Native build takes 10+ minutes → use SKIP_NATIVE_IF_EXISTS=1
  • Symlink errors → git config core.symlinks true && git restore internal/native/cgo/ui

File size: 394 lines (~2 pages), validated against actual build/test commands.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • cdn.jsdelivr.net
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/kvm/kvm/ui/node_modules/.bin/paraglide-js compile --project ./localization/jetKVM.UI.inlang --outdir ./localization/paraglide ux_amd64/vet -p g/grpc/resolver -lang=go1.24 ux_amd64/vet -uns�� g_.a /tmp/go-build2583221049/b103/vet.cfg ux_amd64/vet go1.24.10 to/prf -nolocalimports ux_amd64/vet (dns block)
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/kvm/kvm/ui/node_modules/.bin/paraglide-js compile --project ./localization/jetKVM.UI.inlang --outdir ./localization/paraglide ux_amd64/vet -c=4 t -importcfg ux_amd64/vet -w -Wno-error DQO2/Cd_NwcB7tBpnfIJpDQO2 ux_amd64/vet -c -I /tmp/go-build2583221049/b300/ ux_amd64/vet (dns block)
  • eu.posthog.com
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/kvm/kvm/ui/node_modules/.bin/paraglide-js compile --project ./localization/jetKVM.UI.inlang --outdir ./localization/paraglide ux_amd64/vet -p g/grpc/resolver -lang=go1.24 ux_amd64/vet -uns�� g_.a /tmp/go-build2583221049/b103/vet.cfg ux_amd64/vet go1.24.10 to/prf -nolocalimports ux_amd64/vet (dns block)
    • Triggering command: /usr/local/bin/node node /home/REDACTED/work/kvm/kvm/ui/node_modules/.bin/paraglide-js compile --project ./localization/jetKVM.UI.inlang --outdir ./localization/paraglide ux_amd64/vet -c=4 t -importcfg ux_amd64/vet -w -Wno-error DQO2/Cd_NwcB7tBpnfIJpDQO2 ux_amd64/vet -c -I /tmp/go-build2583221049/b300/ ux_amd64/vet (dns block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

Your task is to "onboard" this repository to Copilot coding agent by adding a .github/copilot-instructions.md file in the repository that contains information describing how a coding agent seeing it for the first time can work most efficiently.

You will do this task only one time per repository and doing a good job can SIGNIFICANTLY improve the quality of the agent's work, so take your time, think carefully, and search thoroughly before writing the instructions.

- Reduce the likelihood of a coding agent pull request getting rejected by the user due to generating code that fails the continuous integration build, fails a validation pipeline, or having misbehavior. - Minimize bash command and build failures. - Allow the agent to complete its task more quickly by minimizing the need for exploration using grep, find, str_replace_editor, and code search tools. - Instructions must be no longer than 2 pages. - Instructions must not be task specific. - Instructions should differ for the VITE/Typescript UI in the /ui directory FROM the go code

Add the following high level details about the codebase to reduce the amount of searching the agent has to do to understand the codebase each time:

- A summary of what the repository does. - High level repository information, such as the size of the repo, the type of the project, the languages, frameworks, or target runtimes in use.

Add information about how to build and validate changes so the agent does not need to search and find it each time.

- For each of build, test, run, lint, and any other scripted step, document the sequence of steps to take to run it successfully as well as the versions of any runtime or build tools used. - Each command should be validated by running it to ensure that it works correctly as well as any preconditions and postconditions. - Try cleaning the repo and environment and running commands in different orders and document errors and and misbehavior observed as well as any steps used to mitigate the problem. - Run the tests and document the order of steps required to run the tests. - Make a change to the codebase. Document any unexpected build issues as well as the workarounds. - Document environment setup steps that seem optional but that you have validated are actually required. - Document the time required for commands that failed due to timing out. - When you find a sequence of commands that work for a particular purpose, document them in detail. - Use language to indicate when something should always be done. For example: "always run npm install before building". - Record any validation steps from documentation.

List key facts about the layout and architecture of the codebase to help the agent find where to make changes with minimal searching.

- A description of the major architectural elements of the project, including the relative paths to the main project files, the location of configuration files for linting, compilation, testing, and preferences. - A description of the checks run prior to check in, including any GitHub workflows, continuous integration builds, or other validation pipelines. - Document the steps so that the agent can replicate these itself. - Any explicit validation steps that the agent can consider to have further confidence in its changes. - Dependencies that aren't obvious from the layout or file structure. - Finally, fill in any remaining space with detailed lists of the following, in order of priority: the list of files in the repo root, the contents of the README, the contents of any key source files, the list of files in the next level down of directories, giving priority to the more structurally important and snippets of code from key source files, such as the one containing the main method. - Perform a comprehensive inventory of the codebase. Search for and view: - README.md, CONTRIBUTING.md, and all other documentation files. - Search the codebase for build steps and indications of workarounds like 'HACK', 'TODO', etc. - All scripts, particularly those pertaining to build and repo or environment setup. - All build and actions pipelines. - All project files. - All configuration and linting files. - For each file: - think: are the contents or the existence of the file information that the coding agent will need to implement, build, test, validate, or demo a code change? - If yes: - Document the command or information in detail. - Explicitly indicate which commands work and which do not and the order in which commands should be run. - Document any errors encountered as well as the steps taken to workaround them. - Document any other steps or information that the agent can use to reduce time spent exploring or trying and failing to run bash commands. - Document any errors encountered as well as th...

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

This file provides detailed instructions for Copilot coding agents to work efficiently with the JetKVM repository, including:

- Repository overview and structure
- Critical build requirements (Go 1.24.4+, Node.js 22.20.0)
- Complete build and validation commands for Go backend and TypeScript UI
- GitHub Actions/CI pipeline documentation
- Project structure and key files
- Common issues and solutions (static directory, Node version, symlinks)
- Development workflows for Go and UI changes
- Important conventions and coding standards

The instructions are validated and designed to minimize exploration time and build failures.

Co-authored-by: IDisposable <[email protected]>
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

1 similar comment
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copilot AI changed the title [WIP] Add Copilot instructions for onboarding Add .github/copilot-instructions.md for agent onboarding Dec 9, 2025
Copilot AI requested a review from IDisposable December 9, 2025 21:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants