commit 183e5f0c891bad7244d4fa4e00cc6eaef36a0a9b · kitten.sh/graphqlsp

.changeset/new-chairs-wonder.md

···

       1
       +
       ---

     

       2
       +
       "@0no-co/graphqlsp": patch

     

       3
       +
       ---

     

       4
       +
       

     

       5
       +
       fix case where the hover-information would target the wrong TypeScript node by one character

+2 -2

packages/graphqlsp/src/ast/token.ts

···

       22
        
       

     

       23
        
         let foundToken: Token | undefined = undefined;

     

       24
        
         for (let line = 0; line < input.length; line++) {

     

       25
       -
           const lPos = cPos;

     

       26
        
           const stream = new CharacterStream(input[line] + '\n');

     

       27
        
           while (!stream.eol()) {

     

       28
        
             const token = parser.token(stream, state);

     

       29
        
             const string = stream.current();

     

       30
        
       

     

       31
        
             if (

     

       32
       -
               lPos + stream.getStartOfToken() <= cursorPosition &&

     

       33
        
               lPos + stream.getCurrentPosition() >= cursorPosition

     

       34
        
             ) {

     

       35
        
               foundToken = {

···

       22
        
       

     

       23
        
         let foundToken: Token | undefined = undefined;

     

       24
        
         for (let line = 0; line < input.length; line++) {

     

       25
       +
           const lPos = cPos - 1;

     

       26
        
           const stream = new CharacterStream(input[line] + '\n');

     

       27
        
           while (!stream.eol()) {

     

       28
        
             const token = parser.token(stream, state);

     

       29
        
             const string = stream.current();

     

       30
        
       

     

       31
        
             if (

     

       32
       +
               lPos + stream.getStartOfToken() + 1 <= cursorPosition &&

     

       33
        
               lPos + stream.getCurrentPosition() >= cursorPosition

     

       34
        
             ) {

     

       35
        
               foundToken = {

+10

test/e2e/fixture-project-tada/fixtures/fragment.ts

···

       10
        
         }

     

       11
        
       `);

     

       12
        
       

     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       13
        
       export const Pokemon = () => {};

···

       10
        
         }

     

       11
        
       `);

     

       12
        
       

     

       13
       +
       // prettier-ignore

     

       14
       +
       export const Regression190 = graphql(`

     

       15
       +
       fragment pokemonFields on Pokemon {

     

       16
       +
         id

     

       17
       +
         name

     

       18
       +
         fleeRate

     

       19
       +
         

     

       20
       +
       }

     

       21
       +
       `);

     

       22
       +
       

     

       23
        
       export const Pokemon = () => {};

test/e2e/fixture-project/fixtures/simple.ts

···

       9
        
         }

     

       10
        
       `;

     

       11
        
       

     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       12
        
       const sql = (x: string | TemplateStringsArray) => x;

     

       13
        
       const x = sql`'{}'`;

···

       9
        
         }

     

       10
        
       `;

     

       11
        
       

     

       12
       +
       const Regression190 = gql`

     

       13
       +
       query AllPosts {

     

       14
       +
         

     

       15
       +
       }

     

       16
       +
       `;

     

       17
       +
       

     

       18
        
       const sql = (x: string | TemplateStringsArray) => x;

     

       19
        
       const x = sql`'{}'`;

+74 -2

test/e2e/graphqlsp.test.ts

···

       109
        
           ]);

     

       110
        
         }, 7500);

     

       111
        
       

     

       112
       -
         it('Gives quick-info when hovering', async () => {

     

       113
        
           server.send({

     

       114
        
             seq: 9,

     

       115
        
             type: 'request',

     
···

       117
        
             arguments: {

     

       118
        
               file: testFile,

     

       119
        
               line: 5,

     

       120
       -
               offset: 7,

     

       121
        
             },

     

       122
        
           });

     

       123
        
       

     
···

       134
        
           expect(res?.body.documentation).toEqual(

     

       135
        
             `Query.posts: [Post]\n\nList out all posts`

     

       136
        
           );

     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       0
        
       
     

       137
        
         }, 7500);

     

       138
        
       });

···

       109
        
           ]);

     

       110
        
         }, 7500);

     

       111
        
       

     

       112
       +
         it('Gives quick-info when hovering start (#15)', async () => {

     

       113
        
           server.send({

     

       114
        
             seq: 9,

     

       115
        
             type: 'request',

     
···

       117
        
             arguments: {

     

       118
        
               file: testFile,

     

       119
        
               line: 5,

     

       120
       +
               offset: 5,

     

       121
        
             },

     

       122
        
           });

     

       123
        
       

     
···

       134
        
           expect(res?.body.documentation).toEqual(

     

       135
        
             `Query.posts: [Post]\n\nList out all posts`

     

       136
        
           );

     

       137
       +
         }, 7500);

     

       138
       +
       

     

       139
       +
         it('Handles empty line (#190)', async () => {

     

       140
       +
           server.send({

     

       141
       +
             seq: 10,

     

       142
       +
             type: 'request',

     

       143
       +
             command: 'completionInfo',

     

       144
       +
             arguments: {

     

       145
       +
               file: testFile,

     

       146
       +
               line: 14,

     

       147
       +
               offset: 3,

     

       148
       +
               includeExternalModuleExports: true,

     

       149
       +
               includeInsertTextCompletions: true,

     

       150
       +
               triggerKind: 1,

     

       151
       +
             },

     

       152
       +
           });

     

       153
       +
       

     

       154
       +
           await server.waitForResponse(

     

       155
       +
             response =>

     

       156
       +
               response.type === 'response' && response.command === 'completionInfo'

     

       157
       +
           );

     

       158
       +
       

     

       159
       +
           const res = server.responses

     

       160
       +
             .reverse()

     

       161
       +
             .find(

     

       162
       +
               resp => resp.type === 'response' && resp.command === 'completionInfo'

     

       163
       +
             );

     

       164
       +
       

     

       165
       +
           expect(res).toBeDefined();

     

       166
       +
           expect(typeof res?.body.entries).toEqual('object');

     

       167
       +
           const defaultAttrs = { kind: 'var', kindModifiers: 'declare' };

     

       168
       +
           expect(res?.body.entries).toEqual([

     

       169
       +
             {

     

       170
       +
               ...defaultAttrs,

     

       171
       +
               name: 'post',

     

       172
       +
               sortText: '0post',

     

       173
       +
               labelDetails: { detail: ' Post' },

     

       174
       +
             },

     

       175
       +
             {

     

       176
       +
               ...defaultAttrs,

     

       177
       +
               name: 'posts',

     

       178
       +
               sortText: '1posts',

     

       179
       +
               labelDetails: { detail: ' [Post]', description: 'List out all posts' },

     

       180
       +
             },

     

       181
       +
             {

     

       182
       +
               ...defaultAttrs,

     

       183
       +
               name: '__typename',

     

       184
       +
               sortText: '2__typename',

     

       185
       +
               labelDetails: {

     

       186
       +
                 detail: ' String!',

     

       187
       +
                 description: 'The name of the current Object type at runtime.',

     

       188
       +
               },

     

       189
       +
             },

     

       190
       +
             {

     

       191
       +
               ...defaultAttrs,

     

       192
       +
               name: '__schema',

     

       193
       +
               sortText: '3__schema',

     

       194
       +
               labelDetails: {

     

       195
       +
                 detail: ' __Schema!',

     

       196
       +
                 description: 'Access the current type schema of this server.',

     

       197
       +
               },

     

       198
       +
             },

     

       199
       +
             {

     

       200
       +
               ...defaultAttrs,

     

       201
       +
               name: '__type',

     

       202
       +
               sortText: '4__type',

     

       203
       +
               labelDetails: {

     

       204
       +
                 detail: ' __Type',

     

       205
       +
                 description: 'Request the type information of a single type.',

     

       206
       +
               },

     

       207
       +
             },

     

       208
       +
           ]);

     

       209
        
         }, 7500);

     

       210
        
       });

+190

test/e2e/tada.test.ts

···

       367
        
             ]

     

       368
        
           `);

     

       369
        
         }, 30000);

     

       370
       +
       

     

       371
       +
         it('gives quick-info at start of word (#15)', async () => {

     

       372
       +
           server.send({

     

       373
       +
             seq: 11,

     

       374
       +
             type: 'request',

     

       375
       +
             command: 'quickinfo',

     

       376
       +
             arguments: {

     

       377
       +
               file: outfileCombinations,

     

       378
       +
               line: 7,

     

       379
       +
               offset: 5,

     

       380
       +
             },

     

       381
       +
           });

     

       382
       +
       

     

       383
       +
           await server.waitForResponse(

     

       384
       +
             response =>

     

       385
       +
               response.type === 'response' && response.command === 'quickinfo'

     

       386
       +
           );

     

       387
       +
       

     

       388
       +
           const res = server.responses

     

       389
       +
             .reverse()

     

       390
       +
             .find(resp => resp.type === 'response' && resp.command === 'quickinfo');

     

       391
       +
       

     

       392
       +
           expect(res).toBeDefined();

     

       393
       +
           expect(typeof res?.body).toEqual('object');

     

       394
       +
           expect(res?.body.documentation).toEqual(`Pokemon.name: String!`);

     

       395
       +
         }, 30000);

     

       396
       +
       

     

       397
       +
         it('gives suggestions with empty line (#190)', async () => {

     

       398
       +
           server.send({

     

       399
       +
             seq: 12,

     

       400
       +
             type: 'request',

     

       401
       +
             command: 'completionInfo',

     

       402
       +
             arguments: {

     

       403
       +
               file: outfileCombinations,

     

       404
       +
               line: 19,

     

       405
       +
               offset: 3,

     

       406
       +
               includeExternalModuleExports: true,

     

       407
       +
               includeInsertTextCompletions: true,

     

       408
       +
               triggerKind: 1,

     

       409
       +
             },

     

       410
       +
           });

     

       411
       +
       

     

       412
       +
           await server.waitForResponse(

     

       413
       +
             response =>

     

       414
       +
               response.type === 'response' && response.command === 'completionInfo'

     

       415
       +
           );

     

       416
       +
       

     

       417
       +
           const res = server.responses

     

       418
       +
             .reverse()

     

       419
       +
             .find(

     

       420
       +
               resp => resp.type === 'response' && resp.command === 'completionInfo'

     

       421
       +
             );

     

       422
       +
       

     

       423
       +
           expect(res).toBeDefined();

     

       424
       +
           expect(typeof res?.body.entries).toEqual('object');

     

       425
       +
           expect(res?.body.entries).toMatchInlineSnapshot(`

     

       426
       +
             [

     

       427
       +
               {

     

       428
       +
                 "kind": "var",

     

       429
       +
                 "kindModifiers": "declare",

     

       430
       +
                 "labelDetails": {

     

       431
       +
                   "detail": " AttacksConnection",

     

       432
       +
                 },

     

       433
       +
                 "name": "attacks",

     

       434
       +
                 "sortText": "0attacks",

     

       435
       +
               },

     

       436
       +
               {

     

       437
       +
                 "kind": "var",

     

       438
       +
                 "kindModifiers": "declare",

     

       439
       +
                 "labelDetails": {

     

       440
       +
                   "detail": " [EvolutionRequirement]",

     

       441
       +
                 },

     

       442
       +
                 "name": "evolutionRequirements",

     

       443
       +
                 "sortText": "2evolutionRequirements",

     

       444
       +
               },

     

       445
       +
               {

     

       446
       +
                 "kind": "var",

     

       447
       +
                 "kindModifiers": "declare",

     

       448
       +
                 "labelDetails": {

     

       449
       +
                   "detail": " [Pokemon]",

     

       450
       +
                 },

     

       451
       +
                 "name": "evolutions",

     

       452
       +
                 "sortText": "3evolutions",

     

       453
       +
               },

     

       454
       +
               {

     

       455
       +
                 "kind": "var",

     

       456
       +
                 "kindModifiers": "declare",

     

       457
       +
                 "labelDetails": {

     

       458
       +
                   "description": "Likelihood of an attempt to catch a Pokémon to fail.",

     

       459
       +
                   "detail": " Float",

     

       460
       +
                 },

     

       461
       +
                 "name": "fleeRate",

     

       462
       +
                 "sortText": "4fleeRate",

     

       463
       +
               },

     

       464
       +
               {

     

       465
       +
                 "kind": "var",

     

       466
       +
                 "kindModifiers": "declare",

     

       467
       +
                 "labelDetails": {

     

       468
       +
                   "detail": " PokemonDimension",

     

       469
       +
                 },

     

       470
       +
                 "name": "height",

     

       471
       +
                 "sortText": "5height",

     

       472
       +
               },

     

       473
       +
               {

     

       474
       +
                 "kind": "var",

     

       475
       +
                 "kindModifiers": "declare",

     

       476
       +
                 "labelDetails": {

     

       477
       +
                   "detail": " ID!",

     

       478
       +
                 },

     

       479
       +
                 "name": "id",

     

       480
       +
                 "sortText": "6id",

     

       481
       +
               },

     

       482
       +
               {

     

       483
       +
                 "kind": "var",

     

       484
       +
                 "kindModifiers": "declare",

     

       485
       +
                 "labelDetails": {

     

       486
       +
                   "description": "Maximum combat power a Pokémon may achieve at max level.",

     

       487
       +
                   "detail": " Int",

     

       488
       +
                 },

     

       489
       +
                 "name": "maxCP",

     

       490
       +
                 "sortText": "7maxCP",

     

       491
       +
               },

     

       492
       +
               {

     

       493
       +
                 "kind": "var",

     

       494
       +
                 "kindModifiers": "declare",

     

       495
       +
                 "labelDetails": {

     

       496
       +
                   "description": "Maximum health points a Pokémon may achieve at max level.",

     

       497
       +
                   "detail": " Int",

     

       498
       +
                 },

     

       499
       +
                 "name": "maxHP",

     

       500
       +
                 "sortText": "8maxHP",

     

       501
       +
               },

     

       502
       +
               {

     

       503
       +
                 "kind": "var",

     

       504
       +
                 "kindModifiers": "declare",

     

       505
       +
                 "labelDetails": {

     

       506
       +
                   "detail": " String!",

     

       507
       +
                 },

     

       508
       +
                 "name": "name",

     

       509
       +
                 "sortText": "9name",

     

       510
       +
               },

     

       511
       +
               {

     

       512
       +
                 "kind": "var",

     

       513
       +
                 "kindModifiers": "declare",

     

       514
       +
                 "labelDetails": {

     

       515
       +
                   "detail": " [PokemonType]",

     

       516
       +
                 },

     

       517
       +
                 "name": "resistant",

     

       518
       +
                 "sortText": "10resistant",

     

       519
       +
               },

     

       520
       +
               {

     

       521
       +
                 "kind": "var",

     

       522
       +
                 "kindModifiers": "declare",

     

       523
       +
                 "labelDetails": {

     

       524
       +
                   "detail": " [PokemonType]",

     

       525
       +
                 },

     

       526
       +
                 "name": "types",

     

       527
       +
                 "sortText": "11types",

     

       528
       +
               },

     

       529
       +
               {

     

       530
       +
                 "kind": "var",

     

       531
       +
                 "kindModifiers": "declare",

     

       532
       +
                 "labelDetails": {

     

       533
       +
                   "detail": " [PokemonType]",

     

       534
       +
                 },

     

       535
       +
                 "name": "weaknesses",

     

       536
       +
                 "sortText": "12weaknesses",

     

       537
       +
               },

     

       538
       +
               {

     

       539
       +
                 "kind": "var",

     

       540
       +
                 "kindModifiers": "declare",

     

       541
       +
                 "labelDetails": {

     

       542
       +
                   "detail": " PokemonDimension",

     

       543
       +
                 },

     

       544
       +
                 "name": "weight",

     

       545
       +
                 "sortText": "13weight",

     

       546
       +
               },

     

       547
       +
               {

     

       548
       +
                 "kind": "var",

     

       549
       +
                 "kindModifiers": "declare",

     

       550
       +
                 "labelDetails": {

     

       551
       +
                   "description": "The name of the current Object type at runtime.",

     

       552
       +
                   "detail": " String!",

     

       553
       +
                 },

     

       554
       +
                 "name": "__typename",

     

       555
       +
                 "sortText": "14__typename",

     

       556
       +
               },

     

       557
       +
             ]

     

       558
       +
           `);

     

       559
       +
         }, 30000);

     

       560
        
       });