add normalization to pypi package keys

app/models/release_package.rb

@@ -32,6 +32,10 @@ class ReleasePackage < ApplicationRecord
 
   accepts_nested_attributes_for :engine
 
+  # see: https://peps.python.org/pep-0503/#normalized-names
+  before_save -> { self.key = key.gsub(/[-_.]+/, '-') },
+    if: -> { pypi? && key_changed? }
+
   validates :product,
     scope: { by: :account_id }
 
@@ -103,6 +107,10 @@ class ReleasePackage < ApplicationRecord
   scope :raw,            ->     { for_engine_key('raw') }
   scope :rubygems,       ->     { for_engine_key('rubygems') }
 
+  delegate :pypi?, :tauri?, :raw?, :rubygems?,
+    to: :engine,
+    allow_nil: true
+
   def engine_id? = release_engine_id?
   def engine_id  = release_engine_id
   def engine_id=(id)

spec/models/release_package_spec.rb

@@ -51,4 +51,16 @@
       end
     end
   end
+
+  it 'should normalize key for pypi package' do
+    package = create(:package, :pypi, key: 'a_test.key', product:, account:)
+
+    expect(package.key).to eq 'a-test-key'
+  end
+
+  it 'should not normalize key for package' do
+    package = create(:package, key: 'a_test.key', product:, account:)
+
+    expect(package.key).to eq 'a_test.key'
+  end
 end