Skip to content

Add KTL AI instruction guide for architecture and conventions#583

Open
Copilot wants to merge 2 commits intodevfrom
copilot/create-ktl-ai-instructions-md
Open

Add KTL AI instruction guide for architecture and conventions#583
Copilot wants to merge 2 commits intodevfrom
copilot/create-ktl-ai-instructions-md

Conversation

Copy link
Contributor

Copilot AI commented Feb 12, 2026

A comprehensive agent guide was requested to document KTL class structure and coding conventions, including JS conversion rules, jQuery exceptions, and guidance to reuse existing KTL utilities. This PR adds that focused reference.

  • New guidance doc
    • KTL_AI_Instructions.md describing KTL singleton/module patterns, keyword usage, and CSS naming/organization
  • JavaScript conventions
    • vanilla JS preference with explicit jQuery exceptions for Knack/Chosen events and existing jQuery functions
    • alert/confirm replacement guidance via ktl.core.selectOption()
  • Reuse and extension
    • directs new reusable logic into existing ktl.* modules and discourages new globals

Example snippet:

this.core = (function () {
    return {
        myFn() { /* ... */ }
    };
})();

💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: CSWinnall <10554243+CSWinnall@users.noreply.github.com>
Copilot AI changed the title [WIP] Add comprehensive KTL_AI_Instructions for agents Add KTL AI instruction guide for architecture and conventions Feb 12, 2026
Copilot AI requested a review from CSWinnall February 12, 2026 22:11
@CSWinnall CSWinnall marked this pull request as ready for review February 12, 2026 22:25
@CSWinnall CSWinnall requested review from cortexrd and removed request for CSWinnall February 13, 2026 22:15
@CSWinnall
Copy link
Collaborator

This is just a draft happy for ammendments to be made @cortexrd

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.

2 participants