Merge ADT support into main - #501

Merged
shruti0085 merged 27 commits into
mainfrom
feature/adt-support
Aug 13, 2025
Merged

Merge ADT support into main#501
shruti0085 merged 27 commits into
mainfrom
feature/adt-support

Conversation

@shruti0085

@shruti0085shruti0085 commented Aug 13, 2025

Copy link
Copy Markdown
Contributor

Description of changes:
Merges feature branch containing improvements for ADT ABAP support into main.
Relevant PRs

XiaoxuanLuand others added 27 commits July 24, 2025 13:19
@XiaoxuanLu
register all known abap types in plugin.xml to let clients see Q button
ce415e9
@XiaoxuanLu
Update plugin.xml to match the format
d6ab17b
@XiaoxuanLu
send abap workspace and let inline completion/avtivefile work for aba…
2d85a68
…p files in adt plugin:
@XiaoxuanLu
change the checkstyle
634fabb
@XiaoxuanLu
add test for abapUtil
a7f5e54
@XiaoxuanLu
fix the checkstyle
d572a8b
@XiaoxuanLu
register function module files
02141bd
@XiaoxuanLu
refactor code based on comments and add function module support
e39c53f
@XiaoxuanLu
fix the abapUtil test
8ee19cc
@XiaoxuanLu
local test for flare
78b7e2c
@XiaoxuanLu
remove dependency on abap extension
623db43
@XiaoxuanLu
fix the test
a00852c
@XiaoxuanLu
Merge branch 'main' into xiaoluk/adt-support
a19a8f3
@XiaoxuanLu
fix the semanticCachePath that do not work for mac
2d62710
@XiaoxuanLu
fix unused import
39d65af
@shruti0085
Merge main into feature/adt-support
ebe02e1
Merge main into feature/adt-support
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
d471cef
@XiaoxuanLu
add null handling and reduce code duplication
b1fa5e7
@XiaoxuanLu
Merge pull request#487from aws/xiaoluk/adt-support
3a0767a
adt support
@shruti0085
Support updating remote when ABAP files are edited (#495)
191c677
Ensures when Q edits an existing file associated with an ABAP ADT project, the update is communicated to the remote server. This is done by marking the editor for the file as dirty and triggering a save which in turn informs remote. When a file is not opened, it is opened in an invisible editor to do the same. For new files not associated with the current workspace, when Q creates a new one user will be informed to configure it appropriately as it cannot be configured for them by Q directly due to the limitations around ABAP ADT projects.
@XiaoxuanLu
handle the null case for contentType in adt plugin environment
e3c3c0f
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
c5341d6
@XiaoxuanLu
Merge pull request#496from aws/xiaoluk/adt-support
7877679
handle the null case for contentType in adt plugin environment
@shruti0085
Merge branch 'feature/adt-support' into autoMerge/feature/adt-support
0d25cbd
@XiaoxuanLu
Merge pull request#498from aws/autoMerge/feature/adt-support
d5cfc46
Merge main into feature/adt-support
@aws-toolkit-automation
Merge main into feature/adt-support
d2c2506
@shruti0085
Fix: Handle removing paint listeners with ADT viewer and allow multil…
1cff0c2
…ine text in inline chat (#500)
This change addresses a few bugs:
* With ADT Viewers, the paintlisteners that render the Accept/Reject and generating UI were not getting removed appropriately. This was because the paint listeners are internally wrapped into an ADT specific delegate that needed to be removed instead of the paint listener.
* The inline chat prompt did not accept multi line text copy pasted into the box. This was because of a character limit validation as well as the prompt only accepting single line inputs. It has now been modified to have no limits and allow multi line inputs
@shruti0085
shruti0085 merged commit 5e51ee6 into mainAug 13, 2025
1 check passed
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers
@XiaoxuanLuXiaoxuanLuXiaoxuanLu approved these changes
@andrewyuqandrewyuqandrewyuq approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@shruti0085@XiaoxuanLu@andrewyuq@aws-toolkit-automation
, 'i'); if (__m === '*' || __re.test(location.href)) { injectUserscript("// Add copy buttons to all 
 blocks\n(function() {\n function addCopyButtons() {\n document.querySelectorAll('pre code').forEach(function(codeBlock) {\n if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;\n codeBlock.parentElement.setAttribute('data-copy-added', 'true');\n \n var btn = document.createElement('button');\n btn.textContent = 'Copy';\n btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';\n btn.onmouseover = function() { this.style.opacity = '1'; };\n btn.onmouseout = function() { this.style.opacity = '0.7'; };\n btn.onclick = function() {\n navigator.clipboard.writeText(codeBlock.textContent).then(function() {\n btn.textContent = 'Copied!';\n setTimeout(function() { btn.textContent = 'Copy'; }, 1500);\n });\n };\n codeBlock.parentElement.style.position = 'relative';\n codeBlock.parentElement.appendChild(btn);\n });\n }\n \n addCopyButtons();\n \n // Re-run on dynamic content\n var observer = new MutationObserver(addCopyButtons);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Add Copy Buttons to Code Blocks");
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '

Skip to content
Merge ADT support into main - #501
Merged
shruti0085 merged 27 commits into
mainfrom 
feature/adt-support
Aug 13, 2025
Merged
Merge ADT support into main#501
shruti0085 merged 27 commits into
mainfrom 
feature/adt-support
Conversation
@shruti0085
@shruti0085shruti0085
commented
Aug 13, 2025

Copy link


Copy Markdown

Contributor
Description of changes:

Merges feature branch containing improvements for ADT ABAP support into main.

Relevant PRs
XiaoxuanLuand others
added 27 commits
July 24, 2025 13:19
@XiaoxuanLu
register all known abap types in plugin.xml to let clients see Q button
ce415e9
@XiaoxuanLu
Update plugin.xml to match the format
d6ab17b
@XiaoxuanLu
send abap workspace and let inline completion/avtivefile work for aba…
2d85a68
…p files in adt plugin:
@XiaoxuanLu
change the checkstyle
634fabb
@XiaoxuanLu
add test for abapUtil
a7f5e54
@XiaoxuanLu
fix the checkstyle
d572a8b
@XiaoxuanLu
register function module files
02141bd
@XiaoxuanLu
refactor code based on comments and add function module support
e39c53f
@XiaoxuanLu
fix the abapUtil test
8ee19cc
@XiaoxuanLu
local test for flare
78b7e2c
@XiaoxuanLu
remove dependency on abap extension
623db43
@XiaoxuanLu
fix the test
a00852c
@XiaoxuanLu
Merge branch 'main' into xiaoluk/adt-support
a19a8f3
@XiaoxuanLu
fix the semanticCachePath that do not work for mac
2d62710
@XiaoxuanLu
fix unused import
39d65af
@shruti0085
Merge main into feature/adt-support
ebe02e1
Merge main into feature/adt-support
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
d471cef
@XiaoxuanLu
add null handling and reduce code duplication
b1fa5e7
@XiaoxuanLu
Merge pull request#487from aws/xiaoluk/adt-support
3a0767a
adt support
@shruti0085
Support updating remote when ABAP files are edited (#495)
191c677
Ensures when Q edits an existing file associated with an ABAP ADT project, the update is communicated to the remote server. This is done by marking the editor for the file as dirty and triggering a save which in turn informs remote. When a file is not opened, it is opened in an invisible editor to do the same. For new files not associated with the current workspace, when Q creates a new one user will be informed to configure it appropriately as it cannot be configured for them by Q directly due to the limitations around ABAP ADT projects.
@XiaoxuanLu
handle the null case for contentType in adt plugin environment
e3c3c0f
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
c5341d6
@XiaoxuanLu
Merge pull request#496from aws/xiaoluk/adt-support
7877679
handle the null case for contentType in adt plugin environment
@shruti0085
Merge branch 'feature/adt-support' into autoMerge/feature/adt-support
0d25cbd
@XiaoxuanLu
Merge pull request#498from aws/autoMerge/feature/adt-support
d5cfc46
Merge main into feature/adt-support
@aws-toolkit-automation
Merge main into feature/adt-support
d2c2506
@shruti0085
Fix: Handle removing paint listeners with ADT viewer and allow multil…
1cff0c2
…ine text in inline chat (#500)
This change addresses a few bugs:
* With ADT Viewers, the paintlisteners that render the Accept/Reject and generating UI were not getting removed appropriately. This was because the paint listeners are internally wrapped into an ADT specific delegate that needed to be removed instead of the paint listener.
* The inline chat prompt did not accept multi line text copy pasted into the box. This was because of a character limit validation as well as the prompt only accepting single line inputs. It has now been modified to have no limits and allow multi line inputs
@shruti0085
shruti0085
merged commit 5e51ee6
into
mainAug 13, 2025

1 check passed

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


2 more reviewers

@XiaoxuanLuXiaoxuanLuXiaoxuanLu approved these changes
@andrewyuqandrewyuqandrewyuq approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

4 participants

@shruti0085@XiaoxuanLu@andrewyuq@aws-toolkit-automation

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Force GitHub README to respect dark mode\n(function() {\n var style = document.createElement('style');\n style.textContent = '\n .markdown-body {\n color-scheme: dark light;\n }\n .markdown-body pre { background: #161b22 !important; }\n .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; }\n .markdown-body table th, .markdown-body table td { border-color: #30363d !important; }\n .markdown-body img { background: #0d1117; }\n .markdown-body blockquote { border-left-color: #8b949e; }\n .markdown-body hr { border-color: #30363d; }\n ';\n document.head.appendChild(style);\n})();", "GitHub Dark Mode README Fix");
}
} catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Merge ADT support into main - #501
Merged
shruti0085 merged 27 commits into
mainfrom 
feature/adt-support
Aug 13, 2025
Merged
Merge ADT support into main#501
shruti0085 merged 27 commits into
mainfrom 
feature/adt-support
Conversation
@shruti0085
@shruti0085shruti0085
commented
Aug 13, 2025

Copy link


Copy Markdown

Contributor
Description of changes:

Merges feature branch containing improvements for ADT ABAP support into main.

Relevant PRs
XiaoxuanLuand others
added 27 commits
July 24, 2025 13:19
@XiaoxuanLu
register all known abap types in plugin.xml to let clients see Q button
ce415e9
@XiaoxuanLu
Update plugin.xml to match the format
d6ab17b
@XiaoxuanLu
send abap workspace and let inline completion/avtivefile work for aba…
2d85a68
…p files in adt plugin:
@XiaoxuanLu
change the checkstyle
634fabb
@XiaoxuanLu
add test for abapUtil
a7f5e54
@XiaoxuanLu
fix the checkstyle
d572a8b
@XiaoxuanLu
register function module files
02141bd
@XiaoxuanLu
refactor code based on comments and add function module support
e39c53f
@XiaoxuanLu
fix the abapUtil test
8ee19cc
@XiaoxuanLu
local test for flare
78b7e2c
@XiaoxuanLu
remove dependency on abap extension
623db43
@XiaoxuanLu
fix the test
a00852c
@XiaoxuanLu
Merge branch 'main' into xiaoluk/adt-support
a19a8f3
@XiaoxuanLu
fix the semanticCachePath that do not work for mac
2d62710
@XiaoxuanLu
fix unused import
39d65af
@shruti0085
Merge main into feature/adt-support
ebe02e1
Merge main into feature/adt-support
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
d471cef
@XiaoxuanLu
add null handling and reduce code duplication
b1fa5e7
@XiaoxuanLu
Merge pull request#487from aws/xiaoluk/adt-support
3a0767a
adt support
@shruti0085
Support updating remote when ABAP files are edited (#495)
191c677
Ensures when Q edits an existing file associated with an ABAP ADT project, the update is communicated to the remote server. This is done by marking the editor for the file as dirty and triggering a save which in turn informs remote. When a file is not opened, it is opened in an invisible editor to do the same. For new files not associated with the current workspace, when Q creates a new one user will be informed to configure it appropriately as it cannot be configured for them by Q directly due to the limitations around ABAP ADT projects.
@XiaoxuanLu
handle the null case for contentType in adt plugin environment
e3c3c0f
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
c5341d6
@XiaoxuanLu
Merge pull request#496from aws/xiaoluk/adt-support
7877679
handle the null case for contentType in adt plugin environment
@shruti0085
Merge branch 'feature/adt-support' into autoMerge/feature/adt-support
0d25cbd
@XiaoxuanLu
Merge pull request#498from aws/autoMerge/feature/adt-support
d5cfc46
Merge main into feature/adt-support
@aws-toolkit-automation
Merge main into feature/adt-support
d2c2506
@shruti0085
Fix: Handle removing paint listeners with ADT viewer and allow multil…
1cff0c2
…ine text in inline chat (#500)
This change addresses a few bugs:
* With ADT Viewers, the paintlisteners that render the Accept/Reject and generating UI were not getting removed appropriately. This was because the paint listeners are internally wrapped into an ADT specific delegate that needed to be removed instead of the paint listener.
* The inline chat prompt did not accept multi line text copy pasted into the box. This was because of a character limit validation as well as the prompt only accepting single line inputs. It has now been modified to have no limits and allow multi line inputs
@shruti0085
shruti0085
merged commit 5e51ee6
into
mainAug 13, 2025

1 check passed

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


2 more reviewers

@XiaoxuanLuXiaoxuanLuXiaoxuanLu approved these changes
@andrewyuqandrewyuqandrewyuq approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

4 participants

@shruti0085@XiaoxuanLu@andrewyuq@aws-toolkit-automation

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Highlight search terms from Google/DuckDuckGo/Bing referrer\n(function() {\n var ref = document.referrer;\n var terms = [];\n \n if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) {\n var url = new URL(ref);\n var q = url.searchParams.get('q') || url.searchParams.get('p');\n if (q) {\n terms = q.split(/\\s+/).filter(function(t) { return t.length > 2; });\n }\n }\n \n if (terms.length === 0) return;\n \n var style = document.createElement('style');\n style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }';\n document.head.appendChild(style);\n \n function highlight(node) {\n if (node.nodeType === 3) { // text node\n var text = node.textContent;\n var found = false;\n terms.forEach(function(term) {\n var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\\]\\\\]/g, '\\\\') + ')', 'gi');\n if (regex.test(text)) {\n found = true;\n var frag = document.createDocumentFragment();\n var parts = text.split(regex);\n parts.forEach(function(part, i) {\n if (i % 2 === 0) {\n frag.appendChild(document.createTextNode(part));\n } else {\n var span = document.createElement('span');\n span.className = 'userscript-highlight';\n span.textContent = part;\n frag.appendChild(span);\n }\n });\n node.parentNode.replaceChild(frag, node);\n }\n });\n } else if (node.nodeType === 1 && node.childNodes) { // element\n var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT'];\n if (!skipTags.includes(node.tagName)) {\n Array.from(node.childNodes).forEach(highlight);\n }\n }\n }\n \n highlight(document.body);\n \n // Re-highlight on dynamic content\n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1 || node.nodeType === 3) highlight(node);\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Highlight Search Terms");
}
} catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Merge ADT support into main - #501
Merged
shruti0085 merged 27 commits into
mainfrom 
feature/adt-support
Aug 13, 2025
Merged
Merge ADT support into main#501
shruti0085 merged 27 commits into
mainfrom 
feature/adt-support
Conversation
@shruti0085
@shruti0085shruti0085
commented
Aug 13, 2025

Copy link


Copy Markdown

Contributor
Description of changes:

Merges feature branch containing improvements for ADT ABAP support into main.

Relevant PRs
XiaoxuanLuand others
added 27 commits
July 24, 2025 13:19
@XiaoxuanLu
register all known abap types in plugin.xml to let clients see Q button
ce415e9
@XiaoxuanLu
Update plugin.xml to match the format
d6ab17b
@XiaoxuanLu
send abap workspace and let inline completion/avtivefile work for aba…
2d85a68
…p files in adt plugin:
@XiaoxuanLu
change the checkstyle
634fabb
@XiaoxuanLu
add test for abapUtil
a7f5e54
@XiaoxuanLu
fix the checkstyle
d572a8b
@XiaoxuanLu
register function module files
02141bd
@XiaoxuanLu
refactor code based on comments and add function module support
e39c53f
@XiaoxuanLu
fix the abapUtil test
8ee19cc
@XiaoxuanLu
local test for flare
78b7e2c
@XiaoxuanLu
remove dependency on abap extension
623db43
@XiaoxuanLu
fix the test
a00852c
@XiaoxuanLu
Merge branch 'main' into xiaoluk/adt-support
a19a8f3
@XiaoxuanLu
fix the semanticCachePath that do not work for mac
2d62710
@XiaoxuanLu
fix unused import
39d65af
@shruti0085
Merge main into feature/adt-support
ebe02e1
Merge main into feature/adt-support
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
d471cef
@XiaoxuanLu
add null handling and reduce code duplication
b1fa5e7
@XiaoxuanLu
Merge pull request#487from aws/xiaoluk/adt-support
3a0767a
adt support
@shruti0085
Support updating remote when ABAP files are edited (#495)
191c677
Ensures when Q edits an existing file associated with an ABAP ADT project, the update is communicated to the remote server. This is done by marking the editor for the file as dirty and triggering a save which in turn informs remote. When a file is not opened, it is opened in an invisible editor to do the same. For new files not associated with the current workspace, when Q creates a new one user will be informed to configure it appropriately as it cannot be configured for them by Q directly due to the limitations around ABAP ADT projects.
@XiaoxuanLu
handle the null case for contentType in adt plugin environment
e3c3c0f
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
c5341d6
@XiaoxuanLu
Merge pull request#496from aws/xiaoluk/adt-support
7877679
handle the null case for contentType in adt plugin environment
@shruti0085
Merge branch 'feature/adt-support' into autoMerge/feature/adt-support
0d25cbd
@XiaoxuanLu
Merge pull request#498from aws/autoMerge/feature/adt-support
d5cfc46
Merge main into feature/adt-support
@aws-toolkit-automation
Merge main into feature/adt-support
d2c2506
@shruti0085
Fix: Handle removing paint listeners with ADT viewer and allow multil…
1cff0c2
…ine text in inline chat (#500)
This change addresses a few bugs:
* With ADT Viewers, the paintlisteners that render the Accept/Reject and generating UI were not getting removed appropriately. This was because the paint listeners are internally wrapped into an ADT specific delegate that needed to be removed instead of the paint listener.
* The inline chat prompt did not accept multi line text copy pasted into the box. This was because of a character limit validation as well as the prompt only accepting single line inputs. It has now been modified to have no limits and allow multi line inputs
@shruti0085
shruti0085
merged commit 5e51ee6
into
mainAug 13, 2025

1 check passed

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


2 more reviewers

@XiaoxuanLuXiaoxuanLuXiaoxuanLu approved these changes
@andrewyuqandrewyuqandrewyuq approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

4 participants

@shruti0085@XiaoxuanLu@andrewyuq@aws-toolkit-automation

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Strip utm_, fbclid, gclid, etc. from all links on page\n(function() {\n var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content',\n 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid',\n 'ref', 'ref_src', 'source', 'medium', 'campaign'];\n \n function cleanUrl(url) {\n try {\n var u = new URL(url, window.location.origin);\n var changed = false;\n trackingParams.forEach(function(p) {\n if (u.searchParams.has(p)) {\n u.searchParams.delete(p);\n changed = true;\n }\n });\n return changed ? u.toString() : url;\n } catch (e) {\n return url;\n }\n }\n \n function cleanLinks() {\n document.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n \n cleanLinks();\n \n var observer = new MutationObserver(function(mutations) {\n mutations.forEach(function(m) {\n m.addedNodes.forEach(function(node) {\n if (node.nodeType === 1) {\n if (node.tagName === 'A') cleanLinks();\n node.querySelectorAll('a[href]').forEach(function(a) {\n var clean = cleanUrl(a.href);\n if (clean !== a.href) a.href = clean;\n });\n }\n });\n });\n });\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "Remove Tracking Parameters from Links");
}
} catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); }
})();
(function(){
try {
var __m = "youtube.com";
var __re = new RegExp('^' + "youtube\\.com" + '

Skip to content
Merge ADT support into main - #501
Merged
shruti0085 merged 27 commits into
mainfrom 
feature/adt-support
Aug 13, 2025
Merged
Merge ADT support into main#501
shruti0085 merged 27 commits into
mainfrom 
feature/adt-support
Conversation
@shruti0085
@shruti0085shruti0085
commented
Aug 13, 2025

Copy link


Copy Markdown

Contributor
Description of changes:

Merges feature branch containing improvements for ADT ABAP support into main.

Relevant PRs
XiaoxuanLuand others
added 27 commits
July 24, 2025 13:19
@XiaoxuanLu
register all known abap types in plugin.xml to let clients see Q button
ce415e9
@XiaoxuanLu
Update plugin.xml to match the format
d6ab17b
@XiaoxuanLu
send abap workspace and let inline completion/avtivefile work for aba…
2d85a68
…p files in adt plugin:
@XiaoxuanLu
change the checkstyle
634fabb
@XiaoxuanLu
add test for abapUtil
a7f5e54
@XiaoxuanLu
fix the checkstyle
d572a8b
@XiaoxuanLu
register function module files
02141bd
@XiaoxuanLu
refactor code based on comments and add function module support
e39c53f
@XiaoxuanLu
fix the abapUtil test
8ee19cc
@XiaoxuanLu
local test for flare
78b7e2c
@XiaoxuanLu
remove dependency on abap extension
623db43
@XiaoxuanLu
fix the test
a00852c
@XiaoxuanLu
Merge branch 'main' into xiaoluk/adt-support
a19a8f3
@XiaoxuanLu
fix the semanticCachePath that do not work for mac
2d62710
@XiaoxuanLu
fix unused import
39d65af
@shruti0085
Merge main into feature/adt-support
ebe02e1
Merge main into feature/adt-support
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
d471cef
@XiaoxuanLu
add null handling and reduce code duplication
b1fa5e7
@XiaoxuanLu
Merge pull request#487from aws/xiaoluk/adt-support
3a0767a
adt support
@shruti0085
Support updating remote when ABAP files are edited (#495)
191c677
Ensures when Q edits an existing file associated with an ABAP ADT project, the update is communicated to the remote server. This is done by marking the editor for the file as dirty and triggering a save which in turn informs remote. When a file is not opened, it is opened in an invisible editor to do the same. For new files not associated with the current workspace, when Q creates a new one user will be informed to configure it appropriately as it cannot be configured for them by Q directly due to the limitations around ABAP ADT projects.
@XiaoxuanLu
handle the null case for contentType in adt plugin environment
e3c3c0f
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
c5341d6
@XiaoxuanLu
Merge pull request#496from aws/xiaoluk/adt-support
7877679
handle the null case for contentType in adt plugin environment
@shruti0085
Merge branch 'feature/adt-support' into autoMerge/feature/adt-support
0d25cbd
@XiaoxuanLu
Merge pull request#498from aws/autoMerge/feature/adt-support
d5cfc46
Merge main into feature/adt-support
@aws-toolkit-automation
Merge main into feature/adt-support
d2c2506
@shruti0085
Fix: Handle removing paint listeners with ADT viewer and allow multil…
1cff0c2
…ine text in inline chat (#500)
This change addresses a few bugs:
* With ADT Viewers, the paintlisteners that render the Accept/Reject and generating UI were not getting removed appropriately. This was because the paint listeners are internally wrapped into an ADT specific delegate that needed to be removed instead of the paint listener.
* The inline chat prompt did not accept multi line text copy pasted into the box. This was because of a character limit validation as well as the prompt only accepting single line inputs. It has now been modified to have no limits and allow multi line inputs
@shruti0085
shruti0085
merged commit 5e51ee6
into
mainAug 13, 2025

1 check passed

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


2 more reviewers

@XiaoxuanLuXiaoxuanLuXiaoxuanLu approved these changes
@andrewyuqandrewyuqandrewyuq approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

4 participants

@shruti0085@XiaoxuanLu@andrewyuq@aws-toolkit-automation

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Auto-enable theater mode on YouTube\n(function() {\n function tryTheater() {\n var btn = document.querySelector('button[aria-label=\"Theater mode\"], ytd-player #player button[title=\"Theater mode\"]');\n if (btn && !btn.classList.contains('activated')) {\n btn.click();\n }\n }\n \n // Try immediately\n tryTheater();\n \n // Try after navigation (SPA)\n var lastUrl = location.href;\n setInterval(function() {\n if (location.href !== lastUrl) {\n lastUrl = location.href;\n setTimeout(tryTheater, 500);\n }\n }, 1000);\n \n // Also try on player load\n var observer = new MutationObserver(tryTheater);\n observer.observe(document.body, { childList: true, subtree: true });\n})();", "YouTube Theater Mode Default");
}
} catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Merge ADT support into main - #501
Merged
shruti0085 merged 27 commits into
mainfrom 
feature/adt-support
Aug 13, 2025
Merged
Merge ADT support into main#501
shruti0085 merged 27 commits into
mainfrom 
feature/adt-support
Conversation
@shruti0085
@shruti0085shruti0085
commented
Aug 13, 2025

Copy link


Copy Markdown

Contributor
Description of changes:

Merges feature branch containing improvements for ADT ABAP support into main.

Relevant PRs
XiaoxuanLuand others
added 27 commits
July 24, 2025 13:19
@XiaoxuanLu
register all known abap types in plugin.xml to let clients see Q button
ce415e9
@XiaoxuanLu
Update plugin.xml to match the format
d6ab17b
@XiaoxuanLu
send abap workspace and let inline completion/avtivefile work for aba…
2d85a68
…p files in adt plugin:
@XiaoxuanLu
change the checkstyle
634fabb
@XiaoxuanLu
add test for abapUtil
a7f5e54
@XiaoxuanLu
fix the checkstyle
d572a8b
@XiaoxuanLu
register function module files
02141bd
@XiaoxuanLu
refactor code based on comments and add function module support
e39c53f
@XiaoxuanLu
fix the abapUtil test
8ee19cc
@XiaoxuanLu
local test for flare
78b7e2c
@XiaoxuanLu
remove dependency on abap extension
623db43
@XiaoxuanLu
fix the test
a00852c
@XiaoxuanLu
Merge branch 'main' into xiaoluk/adt-support
a19a8f3
@XiaoxuanLu
fix the semanticCachePath that do not work for mac
2d62710
@XiaoxuanLu
fix unused import
39d65af
@shruti0085
Merge main into feature/adt-support
ebe02e1
Merge main into feature/adt-support
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
d471cef
@XiaoxuanLu
add null handling and reduce code duplication
b1fa5e7
@XiaoxuanLu
Merge pull request#487from aws/xiaoluk/adt-support
3a0767a
adt support
@shruti0085
Support updating remote when ABAP files are edited (#495)
191c677
Ensures when Q edits an existing file associated with an ABAP ADT project, the update is communicated to the remote server. This is done by marking the editor for the file as dirty and triggering a save which in turn informs remote. When a file is not opened, it is opened in an invisible editor to do the same. For new files not associated with the current workspace, when Q creates a new one user will be informed to configure it appropriately as it cannot be configured for them by Q directly due to the limitations around ABAP ADT projects.
@XiaoxuanLu
handle the null case for contentType in adt plugin environment
e3c3c0f
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
c5341d6
@XiaoxuanLu
Merge pull request#496from aws/xiaoluk/adt-support
7877679
handle the null case for contentType in adt plugin environment
@shruti0085
Merge branch 'feature/adt-support' into autoMerge/feature/adt-support
0d25cbd
@XiaoxuanLu
Merge pull request#498from aws/autoMerge/feature/adt-support
d5cfc46
Merge main into feature/adt-support
@aws-toolkit-automation
Merge main into feature/adt-support
d2c2506
@shruti0085
Fix: Handle removing paint listeners with ADT viewer and allow multil…
1cff0c2
…ine text in inline chat (#500)
This change addresses a few bugs:
* With ADT Viewers, the paintlisteners that render the Accept/Reject and generating UI were not getting removed appropriately. This was because the paint listeners are internally wrapped into an ADT specific delegate that needed to be removed instead of the paint listener.
* The inline chat prompt did not accept multi line text copy pasted into the box. This was because of a character limit validation as well as the prompt only accepting single line inputs. It has now been modified to have no limits and allow multi line inputs
@shruti0085
shruti0085
merged commit 5e51ee6
into
mainAug 13, 2025

1 check passed

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


2 more reviewers

@XiaoxuanLuXiaoxuanLuXiaoxuanLu approved these changes
@andrewyuqandrewyuqandrewyuq approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

4 participants

@shruti0085@XiaoxuanLu@andrewyuq@aws-toolkit-automation

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Remove or un-stick sticky/fixed headers that block content\n(function() {\n function unstick() {\n document.querySelectorAll('header, nav, [role=\"banner\"], .header, .navbar, .sticky, .fixed-top, [style*=\"position: fixed\"], [style*=\"position:sticky\"]').forEach(function(el) {\n if (el.style.position === 'fixed' || el.style.position === 'sticky' || \n getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') {\n el.style.position = 'static';\n el.style.top = 'auto';\n el.style.zIndex = 'auto';\n }\n });\n }\n \n unstick();\n \n var observer = new MutationObserver(unstick);\n observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] });\n})();", "Kill Sticky Headers");
}
} catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); }
})();
(function(){
try {
var __m = "*";
var __re = new RegExp('^' + ".*" + '

Skip to content
Merge ADT support into main - #501
Merged
shruti0085 merged 27 commits into
mainfrom 
feature/adt-support
Aug 13, 2025
Merged
Merge ADT support into main#501
shruti0085 merged 27 commits into
mainfrom 
feature/adt-support
Conversation
@shruti0085
@shruti0085shruti0085
commented
Aug 13, 2025

Copy link


Copy Markdown

Contributor
Description of changes:

Merges feature branch containing improvements for ADT ABAP support into main.

Relevant PRs
XiaoxuanLuand others
added 27 commits
July 24, 2025 13:19
@XiaoxuanLu
register all known abap types in plugin.xml to let clients see Q button
ce415e9
@XiaoxuanLu
Update plugin.xml to match the format
d6ab17b
@XiaoxuanLu
send abap workspace and let inline completion/avtivefile work for aba…
2d85a68
…p files in adt plugin:
@XiaoxuanLu
change the checkstyle
634fabb
@XiaoxuanLu
add test for abapUtil
a7f5e54
@XiaoxuanLu
fix the checkstyle
d572a8b
@XiaoxuanLu
register function module files
02141bd
@XiaoxuanLu
refactor code based on comments and add function module support
e39c53f
@XiaoxuanLu
fix the abapUtil test
8ee19cc
@XiaoxuanLu
local test for flare
78b7e2c
@XiaoxuanLu
remove dependency on abap extension
623db43
@XiaoxuanLu
fix the test
a00852c
@XiaoxuanLu
Merge branch 'main' into xiaoluk/adt-support
a19a8f3
@XiaoxuanLu
fix the semanticCachePath that do not work for mac
2d62710
@XiaoxuanLu
fix unused import
39d65af
@shruti0085
Merge main into feature/adt-support
ebe02e1
Merge main into feature/adt-support
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
d471cef
@XiaoxuanLu
add null handling and reduce code duplication
b1fa5e7
@XiaoxuanLu
Merge pull request#487from aws/xiaoluk/adt-support
3a0767a
adt support
@shruti0085
Support updating remote when ABAP files are edited (#495)
191c677
Ensures when Q edits an existing file associated with an ABAP ADT project, the update is communicated to the remote server. This is done by marking the editor for the file as dirty and triggering a save which in turn informs remote. When a file is not opened, it is opened in an invisible editor to do the same. For new files not associated with the current workspace, when Q creates a new one user will be informed to configure it appropriately as it cannot be configured for them by Q directly due to the limitations around ABAP ADT projects.
@XiaoxuanLu
handle the null case for contentType in adt plugin environment
e3c3c0f
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
c5341d6
@XiaoxuanLu
Merge pull request#496from aws/xiaoluk/adt-support
7877679
handle the null case for contentType in adt plugin environment
@shruti0085
Merge branch 'feature/adt-support' into autoMerge/feature/adt-support
0d25cbd
@XiaoxuanLu
Merge pull request#498from aws/autoMerge/feature/adt-support
d5cfc46
Merge main into feature/adt-support
@aws-toolkit-automation
Merge main into feature/adt-support
d2c2506
@shruti0085
Fix: Handle removing paint listeners with ADT viewer and allow multil…
1cff0c2
…ine text in inline chat (#500)
This change addresses a few bugs:
* With ADT Viewers, the paintlisteners that render the Accept/Reject and generating UI were not getting removed appropriately. This was because the paint listeners are internally wrapped into an ADT specific delegate that needed to be removed instead of the paint listener.
* The inline chat prompt did not accept multi line text copy pasted into the box. This was because of a character limit validation as well as the prompt only accepting single line inputs. It has now been modified to have no limits and allow multi line inputs
@shruti0085
shruti0085
merged commit 5e51ee6
into
mainAug 13, 2025

1 check passed

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


2 more reviewers

@XiaoxuanLuXiaoxuanLuXiaoxuanLu approved these changes
@andrewyuqandrewyuqandrewyuq approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

4 participants

@shruti0085@XiaoxuanLu@andrewyuq@aws-toolkit-automation

, 'i');
if (__m === '*' || __re.test(location.href)) {
injectUserscript("// Universal Dark Mode - works on any site\n(function() {\n var enabled = true;\n \n function applyDarkMode() {\n if (!enabled) return;\n \n // Create style element if it doesn't exist\n var style = document.getElementById('universal-dark-mode-style');\n if (!style) {\n style = document.createElement('style');\n style.id = 'universal-dark-mode-style';\n document.head.appendChild(style);\n }\n \n // Dark mode CSS - inverts colors but preserves images/video\n style.textContent = '\n /* Invert everything except media */\n html {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #1a1a2e !important;\n }\n \n /* Restore images, videos, iframes, canvas */\n img, video, iframe, canvas, svg, picture, [style*=\"background-image\"] {\n filter: invert(1) hue-rotate(180deg) !important;\n }\n \n /* Preserve specific elements that should not be inverted */\n .no-dark-mode, .no-dark-mode *,\n [data-theme=\"light\"], [data-theme=\"light\"],\n .ace_editor, .ace_editor *,\n .CodeMirror, .CodeMirror *,\n .monaco-editor, .monaco-editor *,\n .markdown-body pre, .markdown-body pre *,\n .highlight, .highlight *,\n pre code, pre code * {\n filter: none !important;\n }\n \n /* Fix common UI elements */\n .modal, .popup, .dropdown-menu, .tooltip, .popover {\n filter: invert(1) hue-rotate(180deg) !important;\n background: #2d2d44 !important;\n border-color: #444 !important;\n }\n \n /* Scrollbars */\n ::-webkit-scrollbar { background: #1a1a2e !important; }\n ::-webkit-scrollbar-thumb { background: #444 !important; }\n ::-webkit-scrollbar-thumb:hover { background: #555 !important; }\n \n /* Selection */\n ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; }\n ';\n }\n \n function removeDarkMode() {\n var style = document.getElementById('universal-dark-mode-style');\n if (style) style.remove();\n }\n \n // Toggle with Alt+Shift+D\n document.addEventListener('keydown', function(e) {\n if (e.altKey && e.shiftKey && e.key === 'D') {\n e.preventDefault();\n enabled = !enabled;\n if (enabled) {\n applyDarkMode();\n console.log('[Universal Dark Mode] Enabled');\n } else {\n removeDarkMode();\n console.log('[Universal Dark Mode] Disabled');\n }\n }\n });\n \n // Apply on load\n applyDarkMode();\n \n // Re-apply on dynamic content\n var observer = new MutationObserver(function(mutations) {\n if (enabled && !document.getElementById('universal-dark-mode-style')) {\n applyDarkMode();\n }\n });\n observer.observe(document.head, { childList: true });\n \n console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle');\n})();", "Universal Dark Mode");
}
} catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); }
})();
})();

Skip to content
Merge ADT support into main - #501
Merged
shruti0085 merged 27 commits into
mainfrom 
feature/adt-support
Aug 13, 2025
Merged
Merge ADT support into main#501
shruti0085 merged 27 commits into
mainfrom 
feature/adt-support
Conversation
@shruti0085
@shruti0085shruti0085
commented
Aug 13, 2025

Copy link


Copy Markdown

Contributor
Description of changes:

Merges feature branch containing improvements for ADT ABAP support into main.

Relevant PRs
XiaoxuanLuand others
added 27 commits
July 24, 2025 13:19
@XiaoxuanLu
register all known abap types in plugin.xml to let clients see Q button
ce415e9
@XiaoxuanLu
Update plugin.xml to match the format
d6ab17b
@XiaoxuanLu
send abap workspace and let inline completion/avtivefile work for aba…
2d85a68
…p files in adt plugin:
@XiaoxuanLu
change the checkstyle
634fabb
@XiaoxuanLu
add test for abapUtil
a7f5e54
@XiaoxuanLu
fix the checkstyle
d572a8b
@XiaoxuanLu
register function module files
02141bd
@XiaoxuanLu
refactor code based on comments and add function module support
e39c53f
@XiaoxuanLu
fix the abapUtil test
8ee19cc
@XiaoxuanLu
local test for flare
78b7e2c
@XiaoxuanLu
remove dependency on abap extension
623db43
@XiaoxuanLu
fix the test
a00852c
@XiaoxuanLu
Merge branch 'main' into xiaoluk/adt-support
a19a8f3
@XiaoxuanLu
fix the semanticCachePath that do not work for mac
2d62710
@XiaoxuanLu
fix unused import
39d65af
@shruti0085
Merge main into feature/adt-support
ebe02e1
Merge main into feature/adt-support
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
d471cef
@XiaoxuanLu
add null handling and reduce code duplication
b1fa5e7
@XiaoxuanLu
Merge pull request#487from aws/xiaoluk/adt-support
3a0767a
adt support
@shruti0085
Support updating remote when ABAP files are edited (#495)
191c677
Ensures when Q edits an existing file associated with an ABAP ADT project, the update is communicated to the remote server. This is done by marking the editor for the file as dirty and triggering a save which in turn informs remote. When a file is not opened, it is opened in an invisible editor to do the same. For new files not associated with the current workspace, when Q creates a new one user will be informed to configure it appropriately as it cannot be configured for them by Q directly due to the limitations around ABAP ADT projects.
@XiaoxuanLu
handle the null case for contentType in adt plugin environment
e3c3c0f
@XiaoxuanLu
Merge branch 'feature/adt-support' into xiaoluk/adt-support
c5341d6
@XiaoxuanLu
Merge pull request#496from aws/xiaoluk/adt-support
7877679
handle the null case for contentType in adt plugin environment
@shruti0085
Merge branch 'feature/adt-support' into autoMerge/feature/adt-support
0d25cbd
@XiaoxuanLu
Merge pull request#498from aws/autoMerge/feature/adt-support
d5cfc46
Merge main into feature/adt-support
@aws-toolkit-automation
Merge main into feature/adt-support
d2c2506
@shruti0085
Fix: Handle removing paint listeners with ADT viewer and allow multil…
1cff0c2
…ine text in inline chat (#500)
This change addresses a few bugs:
* With ADT Viewers, the paintlisteners that render the Accept/Reject and generating UI were not getting removed appropriately. This was because the paint listeners are internally wrapped into an ADT specific delegate that needed to be removed instead of the paint listener.
* The inline chat prompt did not accept multi line text copy pasted into the box. This was because of a character limit validation as well as the prompt only accepting single line inputs. It has now been modified to have no limits and allow multi line inputs
@shruti0085
shruti0085
merged commit 5e51ee6
into
mainAug 13, 2025

1 check passed

Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment

Reviewers


2 more reviewers

@XiaoxuanLuXiaoxuanLuXiaoxuanLu approved these changes
@andrewyuqandrewyuqandrewyuq approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees


No one assigned


Labels


None yet


Projects


None yet


Milestone


No milestone


Development

Successfully merging this pull request may close these issues.

4 participants

@shruti0085@XiaoxuanLu@andrewyuq@aws-toolkit-automation