Skip to content

fix: BuildProperty-Generator and AOT compilation detection #4333

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jamescrosswell merged 22 commits into from
Jul 13, 2025
Merged

fix: BuildProperty-Generator and AOT compilation detection #4333

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
705517b
style: remove blank lines
Flash0ver Jul 9, 2025
1bdd47c
fix: Generator emitting source when Application
Flash0ver Jul 9, 2025
c3d8dd7
ref: make Generator sealed
Flash0ver Jul 9, 2025
11c91ed
fix: emitted Dictionary uses StringComparer.OrdinalIgnoreCase
Flash0ver Jul 9, 2025
936f1c0
ref: change generated namespace
Flash0ver Jul 9, 2025
9a90070
ref: rename generated type
Flash0ver Jul 9, 2025
72b4c1c
ref: rename HintName
Flash0ver Jul 9, 2025
ab6fa45
fix: inconsistent tabs vs spaces
Flash0ver Jul 9, 2025
8b4ccf8
ref: remove file-scoped namespace declaration
Flash0ver Jul 9, 2025
c00e25e
ref: ModuleInitializer requires .NET 5.0
Flash0ver Jul 9, 2025
19284af
fix: generated attribute
Flash0ver Jul 9, 2025
20ac323
ref: enable nullable reference types
Flash0ver Jul 9, 2025
a6860a9
ref: make generated type and member internal
Flash0ver Jul 9, 2025
6b8e22e
docs: add CHANGELOG entry
Flash0ver Jul 9, 2025
9d644c4
Implemented missing logic for AotHelper + some diagnostics
jamescrosswell Jul 10, 2025
8fbade9
Update CHANGELOG.md
jamescrosswell Jul 10, 2025
5210353
ref: append Attribute-suffix for consistency
Flash0ver Jul 10, 2025
3f6a86f
ref: remove redundant attribute
Flash0ver Jul 10, 2025
0344f17
build: delete Sentry.SourceGenerators.targets
Flash0ver Jul 10, 2025
560823f
ref: delete redundant string interpolation
Flash0ver Jul 10, 2025
45dd854
Merge branch 'main' into fix/source-generator
jamescrosswell Jul 11, 2025
9c8451d
Update CHANGELOG.md
jamescrosswell Jul 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: emitted Dictionary uses StringComparer.OrdinalIgnoreCase
  • Loading branch information
@Flash0ver
Flash0ver committed Jul 9, 2025
commit 11c91edacd0714e82ef0ba237c85d119a2415f89
3 changes: 2 additions & 1 deletion src/Sentry.SourceGenerators/BuildPropertySourceGenerator.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,8 @@ public static class BuildVariableInitializer
[global::System.Runtime.CompilerServices.ModuleInitializer]
public static void Initialize()
{
global::Sentry.CompilerServices.BuildProperties.Initialize(new global::System.Collections.Generic.Dictionary<string, string> {
global::Sentry.CompilerServices.BuildProperties.Initialize(new global::System.Collections.Generic.Dictionary<string, string>(global::System.StringComparer.OrdinalIgnoreCase)
{

"""
);
Expand Down
99 changes: 57 additions & 42 deletions ...ourceGenerators.Tests/BuildPropertySourceGeneratorTests.RunResult_BadStrings.verified.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
SyntaxTree: {
FilePath: Sentry.SourceGenerators/Sentry.SourceGenerators.BuildPropertySourceGenerator/__BuildProperties.g.cs,
Encoding: utf-8,
Length: 613,
Length: 670,
HasCompilationUnitRoot: true,
Options: {
LanguageVersion: CSharp12,
Expand All @@ -32,14 +32,15 @@ public static class BuildVariableInitializer
[global::System.Runtime.CompilerServices.ModuleInitializer]
public static void Initialize()
{
global::Sentry.CompilerServices.BuildProperties.Initialize(new global::System.Collections.Generic.Dictionary<string, string> {
global::Sentry.CompilerServices.BuildProperties.Initialize(new global::System.Collections.Generic.Dictionary<string, string>(global::System.StringComparer.OrdinalIgnoreCase)
{
{"My\\Key", "test\\test"},
});
}
}
#endif
,
Length: 613,
Length: 670,
ChecksumAlgorithm: Sha1,
CanBeEmbedded: true,
Container: {},
Expand Down Expand Up @@ -251,97 +252,111 @@ public static class BuildVariableInitializer
{
LineNumber: 15,
Start: 430,
End: 564,
EndIncludingLineBreak: 565,
End: 611,
EndIncludingLineBreak: 612,
Span: {
Start: 430,
Length: 134
Length: 181
},
SpanIncludingLineBreak: {
Start: 430,
Length: 135
Length: 182
}
},
{
LineNumber: 16,
Start: 565,
End: 594,
EndIncludingLineBreak: 595,
Start: 612,
End: 621,
EndIncludingLineBreak: 622,
Span: {
Start: 565,
Start: 612,
Length: 9
},
SpanIncludingLineBreak: {
Start: 612,
Length: 10
}
},
{
LineNumber: 17,
Start: 622,
End: 651,
EndIncludingLineBreak: 652,
Span: {
Start: 622,
Length: 29
},
SpanIncludingLineBreak: {
Start: 565,
Start: 622,
Length: 30
}
},
{
LineNumber: 17,
Start: 595,
End: 600,
EndIncludingLineBreak: 601,
LineNumber: 18,
Start: 652,
End: 657,
EndIncludingLineBreak: 658,
Span: {
Start: 595,
Start: 652,
Length: 5
},
SpanIncludingLineBreak: {
Start: 595,
Start: 652,
Length: 6
}
},
{
LineNumber: 18,
Start: 601,
End: 603,
EndIncludingLineBreak: 604,
LineNumber: 19,
Start: 658,
End: 660,
EndIncludingLineBreak: 661,
Span: {
Start: 601,
Start: 658,
Length: 2
},
SpanIncludingLineBreak: {
Start: 601,
Start: 658,
Length: 3
}
},
{
LineNumber: 19,
Start: 604,
End: 605,
EndIncludingLineBreak: 606,
LineNumber: 20,
Start: 661,
End: 662,
EndIncludingLineBreak: 663,
Span: {
Start: 604,
Start: 661,
Length: 1
},
SpanIncludingLineBreak: {
Start: 604,
Start: 661,
Length: 2
}
},
{
LineNumber: 20,
Start: 606,
End: 612,
EndIncludingLineBreak: 613,
LineNumber: 21,
Start: 663,
End: 669,
EndIncludingLineBreak: 670,
Span: {
Start: 606,
Start: 663,
Length: 6
},
SpanIncludingLineBreak: {
Start: 606,
Start: 663,
Length: 7
}
},
{
LineNumber: 21,
Start: 613,
End: 613,
EndIncludingLineBreak: 613,
LineNumber: 22,
Start: 670,
End: 670,
EndIncludingLineBreak: 670,
Span: {
Start: 613
Start: 670
},
SpanIncludingLineBreak: {
Start: 613
Start: 670
}
}
]
Expand Down
99 changes: 57 additions & 42 deletions ...Generators.Tests/BuildPropertySourceGeneratorTests.RunResult_Publish_AotTrue.verified.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@
SyntaxTree: {
FilePath: Sentry.SourceGenerators/Sentry.SourceGenerators.BuildPropertySourceGenerator/__BuildProperties.g.cs,
Encoding: utf-8,
Length: 610,
Length: 667,
HasCompilationUnitRoot: true,
Options: {
LanguageVersion: CSharp12,
Expand All @@ -32,14 +32,15 @@ public static class BuildVariableInitializer
[global::System.Runtime.CompilerServices.ModuleInitializer]
public static void Initialize()
{
global::Sentry.CompilerServices.BuildProperties.Initialize(new global::System.Collections.Generic.Dictionary<string, string> {
global::Sentry.CompilerServices.BuildProperties.Initialize(new global::System.Collections.Generic.Dictionary<string, string>(global::System.StringComparer.OrdinalIgnoreCase)
{
{"PublishAot", "true"},
});
}
}
#endif
,
Length: 610,
Length: 667,
ChecksumAlgorithm: Sha1,
CanBeEmbedded: true,
Container: {},
Expand Down Expand Up @@ -251,97 +252,111 @@ public static class BuildVariableInitializer
{
LineNumber: 15,
Start: 430,
End: 564,
EndIncludingLineBreak: 565,
End: 611,
EndIncludingLineBreak: 612,
Span: {
Start: 430,
Length: 134
Length: 181
},
SpanIncludingLineBreak: {
Start: 430,
Length: 135
Length: 182
}
},
{
LineNumber: 16,
Start: 565,
End: 591,
EndIncludingLineBreak: 592,
Start: 612,
End: 621,
EndIncludingLineBreak: 622,
Span: {
Start: 565,
Start: 612,
Length: 9
},
SpanIncludingLineBreak: {
Start: 612,
Length: 10
}
},
{
LineNumber: 17,
Start: 622,
End: 648,
EndIncludingLineBreak: 649,
Span: {
Start: 622,
Length: 26
},
SpanIncludingLineBreak: {
Start: 565,
Start: 622,
Length: 27
}
},
{
LineNumber: 17,
Start: 592,
End: 597,
EndIncludingLineBreak: 598,
LineNumber: 18,
Start: 649,
End: 654,
EndIncludingLineBreak: 655,
Span: {
Start: 592,
Start: 649,
Length: 5
},
SpanIncludingLineBreak: {
Start: 592,
Start: 649,
Length: 6
}
},
{
LineNumber: 18,
Start: 598,
End: 600,
EndIncludingLineBreak: 601,
LineNumber: 19,
Start: 655,
End: 657,
EndIncludingLineBreak: 658,
Span: {
Start: 598,
Start: 655,
Length: 2
},
SpanIncludingLineBreak: {
Start: 598,
Start: 655,
Length: 3
}
},
{
LineNumber: 19,
Start: 601,
End: 602,
EndIncludingLineBreak: 603,
LineNumber: 20,
Start: 658,
End: 659,
EndIncludingLineBreak: 660,
Span: {
Start: 601,
Start: 658,
Length: 1
},
SpanIncludingLineBreak: {
Start: 601,
Start: 658,
Length: 2
}
},
{
LineNumber: 20,
Start: 603,
End: 609,
EndIncludingLineBreak: 610,
LineNumber: 21,
Start: 660,
End: 666,
EndIncludingLineBreak: 667,
Span: {
Start: 603,
Start: 660,
Length: 6
},
SpanIncludingLineBreak: {
Start: 603,
Start: 660,
Length: 7
}
},
{
LineNumber: 21,
Start: 610,
End: 610,
EndIncludingLineBreak: 610,
LineNumber: 22,
Start: 667,
End: 667,
EndIncludingLineBreak: 667,
Span: {
Start: 610
Start: 667
},
SpanIncludingLineBreak: {
Start: 610
Start: 667
}
}
]
Expand Down
Loading