platformos-check 0.4.12 → 0.4.13
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +4 -4
- data/CHANGELOG.md +6 -0
- data/CONTRIBUTING.md +20 -18
- data/README.md +108 -57
- data/RELEASING.md +14 -7
- data/TROUBLESHOOTING.md +19 -10
- data/data/platformos_liquid/documentation/filters.json +1 -1
- data/data/platformos_liquid/documentation/latest.json +1 -1
- data/data/platformos_liquid/documentation/tags.json +1 -1
- data/docs/api/check.md +7 -6
- data/docs/api/html_check.md +12 -13
- data/docs/api/liquid_check.md +17 -21
- data/docs/api/yaml_check.md +3 -3
- data/docs/checks/TEMPLATE.md.erb +16 -11
- data/docs/checks/convert_include_to_render.md +29 -13
- data/docs/checks/deprecated_filter.md +5 -9
- data/docs/checks/form_action.md +12 -12
- data/docs/checks/form_authenticity_token.md +21 -15
- data/docs/checks/graphql_in_for_loop.md +15 -13
- data/docs/checks/html_parsing_error.md +12 -12
- data/docs/checks/img_lazy_loading.md +13 -11
- data/docs/checks/img_width_and_height.md +21 -23
- data/docs/checks/include_in_render.md +11 -11
- data/docs/checks/invalid_args.md +11 -11
- data/docs/checks/liquid_tag.md +12 -12
- data/docs/checks/missing_enable_comment.md +7 -7
- data/docs/checks/missing_template.md +14 -13
- data/docs/checks/parse_json_format.md +15 -14
- data/docs/checks/parser_blocking_javascript.md +19 -14
- data/docs/checks/required_layout_object.md +5 -7
- data/docs/checks/space_inside_braces.md +12 -12
- data/docs/checks/syntax_error.md +10 -10
- data/docs/checks/template_length.md +12 -12
- data/docs/checks/translation_files_match.md +10 -11
- data/docs/checks/translation_key_exists.md +10 -11
- data/docs/checks/undefined_object.md +11 -13
- data/docs/checks/unknown_filter.md +11 -11
- data/docs/checks/unreachable_code.md +11 -11
- data/docs/checks/unused_assign.md +11 -11
- data/docs/checks/unused_partial.md +7 -11
- data/docs/checks/valid_yaml.md +11 -11
- data/docs/language_server/how_to_correct_code_with_code_actions_and_execute_command.md +62 -70
- data/lib/platformos_check/language_server/handler.rb +6 -6
- data/lib/platformos_check/language_server/variable_lookup_finder.rb +8 -10
- data/lib/platformos_check/platformos_liquid/documentation.rb +2 -2
- data/lib/platformos_check/version.rb +1 -1
- data/platformos-check.gemspec +1 -1
- metadata +5 -5
@@ -169,8 +169,8 @@ module PlatformosCheck
|
|
169
169
|
|
170
170
|
def on_workspace_did_create_files(_id, params)
|
171
171
|
paths = params[:files]
|
172
|
-
|
173
|
-
|
172
|
+
&.map { |file| file[:uri] }
|
173
|
+
&.map { |uri| file_path(uri) }
|
174
174
|
return unless paths
|
175
175
|
|
176
176
|
paths.each do |path|
|
@@ -184,8 +184,8 @@ module PlatformosCheck
|
|
184
184
|
|
185
185
|
def on_workspace_did_delete_files(_id, params)
|
186
186
|
absolute_paths = params[:files]
|
187
|
-
|
188
|
-
|
187
|
+
&.map { |file| file[:uri] }
|
188
|
+
&.map { |uri| file_path(uri) }
|
189
189
|
|
190
190
|
return unless absolute_paths
|
191
191
|
|
@@ -202,8 +202,8 @@ module PlatformosCheck
|
|
202
202
|
# (which might trigger another platformos_app analysis).
|
203
203
|
def on_workspace_will_rename_files(id, params)
|
204
204
|
relative_paths = params[:files]
|
205
|
-
|
206
|
-
|
205
|
+
&.map { |file| [file[:oldUri], file[:newUri]] }
|
206
|
+
&.map { |(old_uri, new_uri)| [relative_path_from_uri(old_uri), relative_path_from_uri(new_uri)] }
|
207
207
|
return @bridge.send_response(id, nil) unless relative_paths
|
208
208
|
|
209
209
|
relative_paths.each do |(old_path, new_path)|
|
@@ -75,11 +75,10 @@ module PlatformosCheck
|
|
75
75
|
is_liquid_variable = content =~ Liquid::VariableStart
|
76
76
|
is_in_variable_segment = previous_char =~ VARIABLE_LOOKUP_CHARACTERS
|
77
77
|
is_on_blank_variable_lookup_position = content[0..cursor - 1] =~ /[{:,-]\s+$/
|
78
|
-
|
79
|
-
|
80
|
-
|
81
|
-
|
82
|
-
)
|
78
|
+
|
79
|
+
is_liquid_variable && (
|
80
|
+
is_in_variable_segment ||
|
81
|
+
is_on_blank_variable_lookup_position
|
83
82
|
)
|
84
83
|
end
|
85
84
|
|
@@ -162,11 +161,10 @@ module PlatformosCheck
|
|
162
161
|
is_liquid_tag = content.match?(Liquid::TagStart)
|
163
162
|
is_in_variable_segment = markup =~ ENDS_WITH_POTENTIAL_LOOKUP
|
164
163
|
is_on_blank_variable_lookup_position = markup =~ ENDS_WITH_BLANK_POTENTIAL_LOOKUP
|
165
|
-
|
166
|
-
|
167
|
-
|
168
|
-
|
169
|
-
)
|
164
|
+
|
165
|
+
is_liquid_tag && (
|
166
|
+
is_in_variable_segment ||
|
167
|
+
is_on_blank_variable_lookup_position
|
170
168
|
)
|
171
169
|
end
|
172
170
|
|
@@ -22,8 +22,8 @@ module PlatformosCheck
|
|
22
22
|
property_entry = SourceIndex
|
23
23
|
.objects
|
24
24
|
.find { |entry| entry.name == object_name }
|
25
|
-
|
26
|
-
|
25
|
+
&.properties
|
26
|
+
&.find { |prop| prop.name == property_name }
|
27
27
|
|
28
28
|
render_doc(property_entry)
|
29
29
|
end
|
data/platformos-check.gemspec
CHANGED
@@ -28,7 +28,7 @@ Gem::Specification.new do |spec|
|
|
28
28
|
spec.require_paths = ["lib"]
|
29
29
|
|
30
30
|
spec.add_dependency('graphql', '~> 2.0.0')
|
31
|
-
spec.add_dependency('liquid', '
|
31
|
+
spec.add_dependency('liquid', '5.4.0')
|
32
32
|
spec.add_dependency('nokogiri', '>= 1.12')
|
33
33
|
spec.add_dependency('parser', '~> 3')
|
34
34
|
|
metadata
CHANGED
@@ -1,7 +1,7 @@
|
|
1
1
|
--- !ruby/object:Gem::Specification
|
2
2
|
name: platformos-check
|
3
3
|
version: !ruby/object:Gem::Version
|
4
|
-
version: 0.4.
|
4
|
+
version: 0.4.13
|
5
5
|
platform: ruby
|
6
6
|
authors:
|
7
7
|
- Piotr Bliszczyk
|
@@ -10,7 +10,7 @@ authors:
|
|
10
10
|
autorequire:
|
11
11
|
bindir: exe
|
12
12
|
cert_chain: []
|
13
|
-
date: 2024-
|
13
|
+
date: 2024-10-07 00:00:00.000000000 Z
|
14
14
|
dependencies:
|
15
15
|
- !ruby/object:Gem::Dependency
|
16
16
|
name: graphql
|
@@ -30,14 +30,14 @@ dependencies:
|
|
30
30
|
name: liquid
|
31
31
|
requirement: !ruby/object:Gem::Requirement
|
32
32
|
requirements:
|
33
|
-
- -
|
33
|
+
- - '='
|
34
34
|
- !ruby/object:Gem::Version
|
35
35
|
version: 5.4.0
|
36
36
|
type: :runtime
|
37
37
|
prerelease: false
|
38
38
|
version_requirements: !ruby/object:Gem::Requirement
|
39
39
|
requirements:
|
40
|
-
- -
|
40
|
+
- - '='
|
41
41
|
- !ruby/object:Gem::Version
|
42
42
|
version: 5.4.0
|
43
43
|
- !ruby/object:Gem::Dependency
|
@@ -373,7 +373,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
|
|
373
373
|
- !ruby/object:Gem::Version
|
374
374
|
version: '0'
|
375
375
|
requirements: []
|
376
|
-
rubygems_version: 3.5.
|
376
|
+
rubygems_version: 3.5.18
|
377
377
|
signing_key:
|
378
378
|
specification_version: 4
|
379
379
|
summary: A platformOS App Linter
|