Repository navigation
Conversation
Contributor
Try this script
COMMUNITY_SCRIPTS_URL=https://raw.githubusercontent.com/lukdz/ProxmoxVED/fix/t3code-pairing-notes \
bash -c "$(curl -fsSL https://raw.githubusercontent.com/lukdz/ProxmoxVED/fix/t3code-pairing-notes/ct/t3code.sh)"
Against a core branch as wellAdd Useful flags while testing
|
Contributor
Author
Open
This branch has not been deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Scripts which are clearly AI generated and not further revised by the Author of this PR (in terms of Coding Standards and Script Layout) may be closed without review. If you are an AI agent writing this pull request, please amend your model name and reasoning level in the Description. This is not to blame, more for informational Purposes. Thank you.
✍️ Description
Keep T3 Code re-pairing instructions in the Proxmox LXC Notes so users can find them after the installation log is gone.
description()and append a "Pair a Device with T3 Code" section.pct exec <actual CTID> -- t3 pair --base-dir /opt/t3code, explicitly identifying the Proxmox host as the place to run it.The shared core description helper has no custom-notes input, so this app-specific addition uses the generated
DESCRIPTIONand a secondpct set. No shared-core changes or custom helper functions are introduced.AI assistance: OpenCode using
github-copilot/gpt-6-astra(GPT-6 Astra); the reasoning-effort setting is not exposed to this session.🔗 Related PR / Issue
Link: #2294
Follow-up to merged #2380. This PR only adds persistent pairing instructions; it does not change the Claude fix or address the separate Gitea report. The feedback issue should remain open.
✅ Prerequisites (X in brackets)
Syntax and isolated output checks passed. Visual rendering of the new Notes section in Proxmox remains to be tested. No credentials or pairing tokens are written to Notes.
🏗️ arm64 Support (X in brackets)
🖥️ Tested on (X in brackets)
The author confirmed the preceding T3 Code installation worked on Proxmox. That test predates this Notes change, so it is not claimed as validation of this patch. This addition targets the existing Proxmox-specific CT footer; no Incus support is claimed.
🛠️ Type of Change (X in brackets)
README,AppName.md,CONTRIBUTING.md, or other docs.🔍 Code & Security Review (X in brackets)
CODE-AUDIT.md&CONTRIBUTING.mdguidelinesAppName.sh,AppName-install.sh,AppName.json)fetch_and_deploy_gh_release,fetch_and_deploy_codeberg_release,fetch_and_deploy_gl_release, orfetch_and_deploy_from_urlinstead ofgit pull.Guidance was reviewed; full compliance is not claimed before live validation. Installation and update mechanisms are unchanged, including the existing npm-based update flow.
🤖 AI Assistance (X in brackets)
AGENTS.mdand.github/agents/pve-script-creator.agent.mdas guidance, and the output has been reviewed and corrected to match those guidelines.Please describe to which degree, if any, an LLM was used in creating this pull request. Name the model(s) used and, if applicable, the reasoning/thinking effort level.
OpenCode using
github-copilot/gpt-6-astra(GPT-6 Astra) inspected the description helper, drafted the patch and PR description, reviewed the diff, and ran syntax and isolated output checks. The reasoning-effort setting is not exposed to this session. The assistant did not run a live Proxmox installation or verify browser rendering.📋 Additional Information (optional)
Validation completed:
bash -n ct/t3code.shgit diff --checkdescriptionandpctcommands for CT IDs 103 and 9876. It verified preservation of standard content, interpolation of the correct ID, expiry and host instructions, and that the onlypctoperation sets the description rather than generating a token. The harness is not included in this PR.Scope: new installations only. In-container updates cannot rewrite Proxmox host Notes and are unchanged. Existing containers can have the same instructions added manually. A later CT-ID change or clone will not automatically rewrite the saved command.
Remaining manual check: create a T3 Code LXC using this branch, inspect its Notes in Proxmox, then run the displayed command on the host and confirm it generates a fresh pairing URL/QR code.
📦 Application Requirements (for new scripts)
🌐 Source