ExecuteAsClause object has wrong LastTokenIndex value

Open

#120 opened on Feb 28, 2025 0 comments 5 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

Assessment

Difficulty: 3/5
Estimated time: 1-2 days
Newbie friendliness: 45/100

Issue type: Bug
Clarity: Clearly specified
Activity status: Stale
Tech stack: csharp, sql
Domain: compilers

Research direction

Reproduce the SQL sample with ScriptDom 161.9142.1 at compatibility level 150 and inspect ExecuteAsClause.FirstTokenIndex and LastTokenIndex. Trace the parser's token-span handling for EXECUTE AS OWNER and add regression coverage showing LastTokenIndex reaches OWNER; done when the existing literal-user behavior remains correct.

Written by the indexing model from the issue text.

Description

ScriptDom version: 161.9142.1
Compatibility level: 150

Sample code:

CREATE PROCEDURE foo
WITH EXECUTE AS OWNER
AS;

Here for the ExecuteAsClause instance values of FirstTokenIndex and LastTokenIndex are equal and pointing to the token representing word EXECUTE which is wrong: FirstTokenIndex is fine but LastTokenIndex should be pointing to the token representing OWNER keyword.

If EXECUTE AS is specified with user name literal then everything is fine, LastTokenIndex is correct.

ExecuteAsClause.LastTokenIndex should be pointing to the very last word of the clause.

Dominant language: GAP
Stars: 277
Forks: 43
Avg merge: 6d 17h
Merged PRs (30d): 3

Contributor guide

Open the contributing guide

First steps

Read the whole issue, then the project's contributing guide.
Comment on the issue to say you are picking it up — it saves two people doing the same work.
Fork the repository and make your change on a branch.
Open a pull request that references the issue number.

Similar issues

isolated-declarations: TS9025 on an optional parameter property typed `readonly T[]`, which tsc 7 accepts Open

Difficulty 2/5 1-3 hours Newbie friendliness 88/100

oxc-project/oxc#26944 · Opened Sep 22, 2026
perf: a definite-value return type (`--> Nil`, `--> True`) sends every call down the full path (~12 µs vs ~1 µs) Open

todo:perf

Difficulty 2/5 1-3 hours Newbie friendliness 84/100

tokuhirom/mutsu#9074 · Opened Sep 22, 2026
A function-call exponent (`A^sqrt(n)`) emits `pow(A, sqrt)(n)` — valid BNGL that codegen cannot compile Open

bug

Difficulty 2/5 Half a day Newbie friendliness 82/100

lanl/bngsim#657 · Opened Sep 22, 2026
tt_metal/hw/inc/internal/tt-2xx/quasar/overlay/xcustom_test.hpp needs reimplementing Open

Difficulty 2/5 1-3 hours Newbie friendliness 68/100

tenstorrent/tt-metal#57393 · 1 comment · Opened Sep 22, 2026
StUnhex.number() misformats Long.MIN_VALUE as scientific notation instead of a plain integer Open

Difficulty 2/5 1-3 hours Newbie friendliness 86/100

objectionary/eo#8894 · Opened Sep 22, 2026

More Compilers issues